Replacing PR #60 with an updated version to address feedback and send it to the correct branch.
This incorporates my feature addition as well as @pingou's alembic migration code.
Hm, there are two merge commits in this PR? Could you remove them?
Also you used the remote PR procedure for this? :open_mouth:
Looks like I've made a typo: Unicde -> Unicode :)
Ok, appart from that typo it seems to work fine.
There is one item I think we should add (but I can do it in a later PR), it's the ability to claim the badge w/o voting. You have to be on the page where you vote but you don't actually have to vote to get the badge.
rebased onto 3d093f94e04e02b0374d6bd90df75cf15d6896c2
@pingou Commits cleaned up and typo corrected.
I'm not sure how I feel about claiming the badge without casting a ballot. I'm not necessarily opposed, I'm just not sold on it.
I believe the idea was to prevent people from linking who voted on what based on results vs when people were claiming badge or something. To be honest I'm fuzzy on the reason but I remember it was an explicit demand for nuancier. I'm not very much sold on it, but I also don't see the arm.
I have another UI fix (unrelated to this feature) so I was thinking to have the two in one PR, so we can go ahead and get this one in.
What do you think?
Works for me
Pull-Request has been merged by pingou