search

esmesh / allourideas.org

allourideas allows groups to collect and priorize information in an open, democratic, and efficient process.
www.allourideas.org
0 stars 0 forks source link

Merge Visitor into User within allourideas.org #33

Closed esmesh closed 9 years ago

esmesh commented 9 years ago

Merge Visitor into User (all "visitors" must now be logged in as users)

esmesh commented 9 years ago

The original design allowed for votes from anonymous subjects -- called "visitors" in the AOI source code). "Users" were external researchers who created a question. To complicate things, "user" in the underlying "pairwise-api" refers to the client application (i.e. the AOI GUI) and "visitor" is anyone else. As a result, the end votes were tracked against HTTP session IDs, not to specific users.

In order to support filtering, I needed better tracking. To do this, I've removed the concept of a "visitor" to the main AOI GUI (and merged the handful of unique items to the "user" class) and changed the vote tracking to be against user IDs, not session IDs, when data is passed down to the pairwise-api.

The merge and tracking is done. Now I'm just working through the question creation/voting process to make sure that I caught every place where the user id is needed (for example, right now, I'm working making sure the admin user that creates a question is tracked properly and the data stored in the DB).

esmesh commented 9 years ago

Closing this issue with the merge of the refactoring branch into master allouridea.org: c2758c7d78d44d3250ae9c9d6899b46159784e4b pairwise-api: esmesh/pairwise-api@596947aa05d88933efa884f61e0d185f8c684b84

esmesh commented 9 years ago

Any remaining testing/fixes that are consequences of this will be tracked via new issues.