Skip to content
This repository has been archived by the owner on May 28, 2020. It is now read-only.

Add Block Users feature #13

Open
mitchdowney opened this issue Aug 6, 2013 · 5 comments
Open

Add Block Users feature #13

mitchdowney opened this issue Aug 6, 2013 · 5 comments
Assignees
Milestone

Comments

@mitchdowney
Copy link
Member

In the Edit Election menu, the moderators for a specific election need the ability to block users. Users that are blocked should be removed from the candidate list, and should have all of their comments removed that the blocked user posted to candidate profiles within only that election.

In other words, the block users function (for the MVP) should remove all traces of the blocked user only in the election that user is blocked from.

Below is an image from the previous evmini of the blocked users field on the Edit Election page:

image

@ghost ghost assigned scvnc Aug 19, 2013
@mitchdowney
Copy link
Member Author

assigned yew vince since you already started this

@mitchdowney
Copy link
Member Author

This is working on my local runserver, but is currently wonky on www.everyvote.org. For example, if I block Abraham Lincoln and Albert Einstein in the EVU - Student Union - Fall 2013 election, neither Abe or Albert are blocked, but Booker T Washington and Frederick Douglass are blocked.

I suspect what's going wrong has to do with the fact userprofile.id is consistently 10 digits higher than its corresponding user.id. I'm not sure how to fix it though.

@scvnc
Copy link
Member

scvnc commented Sep 5, 2013

I could be user #2 but my userprofile could be #20001 for all I could care.

@mitchdowney
Copy link
Member Author

I cannot reproduce this issue locally with the current repo at https://github.com/everyvote/everyvote_mini/tree/mitch_september

So I pushed the relevant pieces for this issue from mitch_september to the everyvote server, dropped all of the existing database tables, ran syncdb, and populated the app with new data. Now blocked_users is blocking the correct users. Maybe this is fixed? Or maybe this is a timebomb issue waiting to break? In case it appears to be at least temporarily fixed right now, and if it is, this sucka is MVP!

@scvnc
Copy link
Member

scvnc commented Sep 7, 2013

I'm voting on timebomb until we explain the interactions with user and userprofile

@scvnc scvnc reopened this Sep 7, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants