Skip to content

feat: Add top rated section to explore page#5447

Merged
steverydz merged 1 commit intomainfrom
WD-30623-top-rated-explore-page
Nov 11, 2025
Merged

feat: Add top rated section to explore page#5447
steverydz merged 1 commit intomainfrom
WD-30623-top-rated-explore-page

Conversation

@steverydz
Copy link
Copy Markdown
Contributor

@steverydz steverydz commented Nov 3, 2025

Done

  • Adds the "Top rated snaps" section to the explore page
  • Creates a reusable include for snaps which are a numbered list

How to QA

Testing

  • This PR has tests
  • No testing required (explain why): No behavioural changes

Issue / Card

Fixes https://warthogs.atlassian.net/browse/WD-30623

Copilot AI review requested due to automatic review settings November 3, 2025 12:48
@webteam-app
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a "Top rated snaps" section to the explore page. The changes fetch top-rated snaps from the recommendations API and display them in a new section on the explore page, positioned between the "Updated snaps" and "Popular snaps" sections.

Key Changes

  • Added API call to fetch top-rated snaps from /top_rated endpoint with error handling
  • Created new template _top-rated-snaps.html to display the top 8 rated snaps with rankings
  • Integrated the new section into the explore page layout

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
webapp/store/views.py Added API call to fetch top-rated snaps and pass them to the template
templates/explore/index.html Added conditional rendering of the top-rated snaps section between recent and popular snaps
templates/explore/_top-rated-snaps.html New template displaying top 8 rated snaps with numbering, icons, titles, publishers, and developer badges

Comment thread templates/explore/index.html Outdated
Comment thread templates/explore/_snaps-list.html
Comment thread templates/explore/_top-rated-snaps.html
Comment thread templates/explore/_top-rated-snaps.html
@steverydz steverydz force-pushed the WD-30623-top-rated-explore-page branch 2 times, most recently from 4244c7a to 44ed903 Compare November 3, 2025 15:36
@steverydz steverydz changed the title chore: Add top rated section to explore page feat: Add top rated section to explore page Nov 5, 2025
@steverydz steverydz marked this pull request as draft November 5, 2025 08:43
@steverydz steverydz force-pushed the WD-30623-top-rated-explore-page branch from 44ed903 to c05138b Compare November 5, 2025 08:51
@steverydz steverydz force-pushed the WD-30623-top-rated-explore-page branch from c05138b to 936ccd6 Compare November 11, 2025 11:39
@steverydz steverydz marked this pull request as ready for review November 11, 2025 11:41
@steverydz steverydz merged commit 0add04e into main Nov 11, 2025
13 checks passed
@steverydz steverydz deleted the WD-30623-top-rated-explore-page branch November 11, 2025 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants