search

marckunze / rancor

Ranked Choice voting tool repo
https://rancor.herokuapp.com
Other
0 stars 0 forks source link

Vote link #51

Closed nvh82 closed 9 years ago

nvh82 commented 9 years ago

I threw in a confirmation page to test the vote button, but we definitely don't have to keep this or we can combine a confirmation message on the same page. Any suggestions as to what happens when the user clicks "vote?" Should the results be displayed, a message confirmed, or any other suggestions?

MarkHammondJr commented 9 years ago

Redirected to the results page with a confirmation message added.

marckunze commented 9 years ago

I tend to agree with Mark, that or a confirmation dialog should be a bit simpler. I'm guessing he's looking at Sinatra-Flash for part of that work.

MarkHammondJr commented 9 years ago

You’re guessing correctly.

MarkHammondJr commented 9 years ago

Can someone remove the redirect to confirmation from the poll.js? I was just going to handle the redirect to the results page from sinatra but I don't want to accidentally break something.

MarkHammondJr commented 9 years ago

Vote seems to be working.