Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Backend/group recommendations improvement #1076

Merged
merged 2 commits into from
Dec 23, 2023
Merged

Backend/group recommendations improvement #1076

merged 2 commits into from
Dec 23, 2023

Conversation

zeynep-baydemir
Copy link
Contributor

  • If the user is not authenticated or doesn't have any game or group in the profile, ten groups are recommended.
  • If the user doesn't have any group but has games in the profile, groups related to user's games are recommended.
  • If the user has groups but doesn't have games, groups with same gameId and related gameIds are recommended.
  • If the user has both groups and games in the profile, related to games and groups are recommendded.

@zeynep-baydemir zeynep-baydemir added Type: Feature Issues related to adding new features or functionality to the project. Team: Backend Issues related to backend team labels Dec 23, 2023
@zeynep-baydemir zeynep-baydemir added this to the 3rd Milestone milestone Dec 23, 2023
@zeynep-baydemir zeynep-baydemir self-assigned this Dec 23, 2023
canuzdrn
canuzdrn approved these changes Dec 23, 2023
View reviewed changes
Copy link
Contributor

@canuzdrn canuzdrn left a comment

Choose a reason for hiding this comment

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

Thanks for the changes/improvements, functionality seems aligned with what we have discussed during the Tuesday lab session. Thanks for the work

@canuzdrn canuzdrn merged commit 838c038 into main Dec 23, 2023
@zeynep-baydemir zeynep-baydemir linked an issue Dec 23, 2023 that may be closed by this pull request
Improving Group Recommendations #1044
Closed
@zeynep-baydemir zeynep-baydemir mentioned this pull request Dec 23, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@canuzdrn canuzdrn canuzdrn approved these changes

Assignees

@zeynep-baydemir zeynep-baydemir

Labels
Team: Backend Issues related to backend team Type: Feature Issues related to adding new features or functionality to the project.
Projects
None yet
Milestone
3rd Milestone
Development

Successfully merging this pull request may close these issues.

Improving Group Recommendations
2 participants
@zeynep-baydemir @canuzdrn