cckrk / native-trials Goto Github PK

Create better images for this project.

Currently assigns userID from the "/AuthenticateUser" (this call probably needs to be renamed in the API as well...) cast to an application setting (new instance of "token") this is a bad design and there must be a better way to store who is logged in in session data (if that should even be stored at all...)

Lists are currently being populated through UserIDs in workout-list-view-model.js' load() method.
The load() method passes the UserID to the API as a parameter, which is later used in the SQL stored procedure sp_GetAllItems to match to a particular (unique) routine.

This is horrible implementation and naming. workout-list-view-model needs to pass some number (or list of numbers) that corresponds to the Users currently subscribed routines, not their ID. Current implementation does not even persist data.

This issue should not be closed until load() method passes the correct values and API variable naming is correct.