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.

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

feat(Challenge Page): A page for community challenges #935

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

TTalex
Copy link
Collaborator

@TTalex TTalex commented Oct 5, 2024

What

  • See Community challenges ? #934 for description, and questions
  • This PR is only used to see code diffs for the prototype, it is not ready for merging

Fixes bug(s)

@teolemon
Copy link
Member

@TTalex do you have screenshots of this ?

@raphodn
Copy link
Member

raphodn commented Dec 31, 2024

@TTalex do you have screenshots of this ?

There is 1 in the related issue (#934)

@TTalex TTalex force-pushed the ttalex/challenge-page branch from 396baea to d9f56b6 Compare January 10, 2025 21:41
@TTalex TTalex force-pushed the ttalex/challenge-page branch from d9f56b6 to d702e49 Compare January 10, 2025 21:43
@TTalex
Copy link
Collaborator Author

TTalex commented Jan 11, 2025

Some updates with the upcoming Nutella challenge:

  • Added two tutorial cards leading to multiple proof uploader and Price validation assistant
  • Changed title, dates, example image and tracked categories to match the new challenge
  • Moved most text to translations
  • Added stats on proof uploads
  • Moved page to experiments
  • Make API calls work using feat(Price Stats): Stats grouped by open-prices#675

New Page look:
Capture1

@TTalex TTalex marked this pull request as ready for review January 11, 2025 15:21
@TTalex
Copy link
Collaborator Author

TTalex commented Jan 11, 2025

Update:

  • Removed elements requiring group by stats (Top contributors, user ranking, chart)
  • Reverted change on PriceChart, as it is no longer used
  • Edit filter name from product__categories_tags__contains to product__categories_tags__contains, which will most likely be the way it's added on backend (see Price API: Allow filter on product categories tags open-prices#673)
  • Some mobile friendly UI tweaks

New page look:
Screenshot 2025-01-11 at 17-28-46 Open Prices

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Status: Backlog
Development

Successfully merging this pull request may close these issues.

Community challenges ?
3 participants