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

205 frontpage remains static for long #207

Merged
merged 8 commits into from
Sep 12, 2024
Merged

205 frontpage remains static for long #207

merged 8 commits into from
Sep 12, 2024

Conversation

tfnribeiro
Copy link
Collaborator

@tfnribeiro tfnribeiro commented Sep 12, 2024 •
edited
Loading

Changes

Re-worked the priority given to the different parameters in the query.

  • In the home page, we no longer use the should query to include the user searches, but instead do a small query to find if there are any relevant new articles which are then ordered with the general topic preferences and recency filter. The home page essentially prioritizes Topic > (Recency == Difficulty Level)
  • Extended the Search endpoint to allow users to fine-control their searches. The user can use the UI to specify they'd rather see more relevant content at the cost of less recent articles or difficulty.

The UI changes are implemented in: zeeguu/web#530

@tfnribeiro tfnribeiro linked an issue Sep 12, 2024 that may be closed by this pull request
Frontpage Remains Static for Long #205
Closed
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 12, 2024 •
edited
Loading

Module view diffs:
diff

@tfnribeiro tfnribeiro mentioned this pull request Sep 12, 2024
Merged
mircealungu
mircealungu approved these changes Sep 12, 2024
View reviewed changes
Copy link
Member

@mircealungu mircealungu left a comment

Choose a reason for hiding this comment

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

looks cool. I'll merge because i want to play with it :)

zeeguu/api/endpoints/search.py
"""
This endpoint is used for the preview search and sending notification
emails.
The call prioritizes the recency of articles paried with the difficulty.
Copy link
Member

Choose a reason for hiding this comment

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

what is "paried"?

@mircealungu mircealungu merged commit f6dc329 into master Sep 12, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mircealungu mircealungu mircealungu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Frontpage Remains Static for Long
2 participants
@tfnribeiro @mircealungu