Skip to content

Feat: categorize projects #627

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Dec 29, 2024
Merged

Feat: categorize projects #627

merged 8 commits into from
Dec 29, 2024

Conversation

ZibanPirate
Copy link
Member

@ZibanPirate ZibanPirate commented Dec 29, 2024

test: https://stage.dzcode.io/projects

Categorize each project tag, which can be: solve-algerian-problem, by-algerian or non-categorized if no tags were found.

  • tagged most projects in ./data
  • added tags table
  • added project_tag_relation
  • in cron, link a project with a tag
  • populate and expose tags in GET /projects
  • show tags on the frontend on this order:
    • solve-algerian-problem
    • by-algerian
    • non-categorized

NOTE:
tag table only has id, later on, we will add name_en and name_ar when we have ai-translation feature implemented.

  • Bug fix
  • New feature
  • Other

@github-actions github-actions bot added project Project data Data related changes ( data folder ) api API related changes ( api folder ) models models package labels Dec 29, 2024
@github-actions github-actions bot added web Web related changes ( web folder ) feature Feature labels Dec 29, 2024
@ZibanPirate ZibanPirate marked this pull request as ready for review December 29, 2024 17:00
@ZibanPirate ZibanPirate requested a review from a team December 29, 2024 17:00
@ZibanPirate ZibanPirate changed the title Feat: project category Feat: categorize projects Dec 29, 2024
omdxp
omdxp previously approved these changes Dec 29, 2024
Comment on lines +77 to +81
// const projectsFromDataFolder = (await this.dataService.listProjects()).filter((p) =>
// ["dzcode.io website", "Mishkal", "System Monitor"].includes(p.name),
// );
// or uncomment to skip the cron
// if (Math.random()) return;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better handle this with Node env

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added a todo comment in fbca05e

@ZibanPirate ZibanPirate merged commit 67a223b into main Dec 29, 2024
20 checks passed
@ZibanPirate ZibanPirate deleted the feat/project-category branch December 29, 2024 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api API related changes ( api folder ) data Data related changes ( data folder ) feature Feature models models package project Project web Web related changes ( web folder )
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

3 participants