Finally “finished” my Voting app! Hooray!Some of the feedback for this will be really low hanging fruit, but I’m sooooo tired of working on this project I can’t wait to share it.
It’s not 100% production ready, but it works. There are a few “todos” I know I need to clean it up, like disallow repeat voting, clean up some centering on certain page, etc. If you’d like to help give me any other feedback about how it works or looks I’d appreciate it. Can’t believe how long this took. It’s not perfect but all user stories are someone implemented.
one thing that perplexes me, is why the logged in user doesn’t persist
lots of refactoring that i could do to clean up the logged in options at the top as well. But i’m pretty excited
You should protect routes that should be accessible only to logged-in users.
Do you want anybody to be able to get list of polls? Yes. Don’t protect.
Does anonymous user should be able to delete a poll? No. Protect.
Does anonymous user should be able to post his vote? Yes. Don’t protect.
Does anonymous user should be able to add (post) voting options? No. Protect.
etc.
Your code also doesn’t check if a poll I want to delete belongs to me (even after you’ll protect the delete route, anyone will be able to log in and delete any poll).
working on those todos right now ,but first writing up a blog post (maybe a medium article) about coding this project and things I learned. It may help others, but mostly it helps me consolidate my learning
I think i’ve fixed all the issues you brought up. Would you mind checking it out again if you have time? This was about 10-12 hours of additional work, but well worth the learning.
)
