Skip to content

add tag-based filtering and recommendation sorting - #120

Merged
haniazipser merged 4 commits into
mainfrom
feature-recommendations
May 27, 2026
Merged

add tag-based filtering and recommendation sorting#120
haniazipser merged 4 commits into
mainfrom
feature-recommendations

Conversation

@cssma

@cssma cssma commented May 10, 2026

Copy link
Copy Markdown
Collaborator

📄 Pull Request Description

Please fill out each section below.
The goal is that reviewers can understand everything without reading the code.


🧩 What was changed?

Replaced allergen-only tag system with a full tag system (dietary, allergens, cuisine). Added recommendation-based sorting on the overview page. User preferences (cuisine, dietary, allergens) moved to profile page only. Hidden preferences section for restaurant owners.


💡 Why was it changed?

JUCanEat/documentation#67


⚙️ How was it implemented?

Tags are now grouped into 3 categories: dietary, allergens, cuisine. Restaurants on the overview page are sorted by score based on user preferences. Overview and menu page filters kept simple (vegan, vegetarian, lactose-free, gluten-free).


⚠️ Side Effects or Risks

none


✅ Checklist

  • All 4 sections above are clearly filled out
  • Tests and documentation updated

@cssma
cssma requested a review from kingazm May 17, 2026 11:26
Comment thread app/i18n.ts Outdated
Comment thread app/api/user_service.tsx
Comment thread public/silent-check-sso.html
@cssma
cssma requested a review from kingazm May 20, 2026 07:44

@kingazm kingazm left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please ensure backend merged too before pushing to prod:)

@haniazipser
haniazipser merged commit 5f8348f into main May 27, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants