search

BuidlGuidl / hackathon-starter-kit

https://extensions-hackathon.vercel.app
MIT License
6 stars 1 forks source link

Allow admins to vote submissions #20

Closed damianmarti closed 2 months ago

damianmarti commented 2 months ago

Vote with a score from 1 to 10.

Public votes for now.

localhost_3000_admin

close #19

damianmarti commented 2 months ago

@technophile-04 @rin-st for review...

Pabl0cks commented 2 months ago

I like the 1 to 10 system better than 0 to 10 commented in https://github.com/BuidlGuidl/extensions-hackathon/issues/9 because you clearly see which ones you voted already, and which ones are missing your vote.

Once you vote you cant undo the vote and leave it empty, but I don't see any problem.

damianmarti commented 2 months ago

Nice job Damu! Voting system is simple and does the job 🙌

I left a comment with a warning I was getting.

=============================================

I think we could tweak the UI/UX a bit but we can tackle it later, and we could ask help to @andrealbiac for it. Leaving a couple of notes I took about it:

  • We could show more than 1 submission per row
  • We should force cards to have the same weight
  • Comments could be more compact and with collapse icon

Leaving some screenshots here too.

Cards without same weight Current comments Compact comments idea

Thanks for the review!

Yes, we need to improve how this looks for sure!

damianmarti commented 2 months ago

@Pabl0cks I fixed the warning and I added the option to remove a vote (maybe the user made a mistake) adding an "x" to the left of the stars.

localhost_3000_admin (1)

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
extensions-hackathon ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 16, 2024 2:39pm
carletex commented 2 months ago

Updating the live database! SInce we don't have a migration workflow I have to do it manually everytime we change the database.