Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User management #12

Closed
matthewberryman opened this issue Jan 12, 2016 · 8 comments
Closed

User management #12

matthewberryman opened this issue Jan 12, 2016 · 8 comments
Assignees
Milestone

Comments

@matthewberryman
Copy link
Contributor

  • Allow creation of new users.
  • Password management.
  • Setting roles (read only vs read write).
  • Logout button.
  • Logout period (define and implement).
@matthewberryman
Copy link
Contributor Author

Even before we wrap this up, it would be great to at least have the password field set as a password field (so characters entered appear as asterisks) prior to the demo. @benatwork99

@matthewberryman
Copy link
Contributor Author

The password field is behaving properly this morning for me, across IE11 and Chrome (Mac) and Safari. So just leave it as is.

@benatwork99
Copy link
Contributor

That was me, I committed that yesterday but forgot that issue #12 was in
cognicity-rem-web (commit was to cognicity-rem-server!)

On 5 February 2016 at 08:06, Matthew Berryman [email protected]
wrote:

The password field is behaving properly this morning for me, across IE11
and Chrome (Mac) and Safari. So just leave it as is.


Reply to this email directly or view it on GitHub
#12 (comment)
.

@matthewberryman
Copy link
Contributor Author

Ah, that's why I couldn't find it in commit history for cognicity-rem-web, it was handled by cogncity-rem-server. My bad. I must have tested yesterday before I deployed that set of commits. 👍

@benatwork99
Copy link
Contributor

All good, I referenced issue #12 but also forgot the issue was in another
project so the commit didn't link to the issue properly!

On 5 February 2016 at 09:23, Matthew Berryman [email protected]
wrote:

Ah, that's why I couldn't find it in commit history for cognicity-rem-web,
it was handled by cogncity-rem-server. My bad. I must have tested yesterday
before I deployed that set of commits. [image: 👍]


Reply to this email directly or view it on GitHub
#12 (comment)
.

@benatwork99
Copy link
Contributor

I'm thinking about a simple schema for this - one user table with columns hardcoded to the roles. E.g., the user table may have these columns:
id, username, password, editor, admin

I thought we could have admin as a role as well, meaning a user who has access to the user management section.

Those are the only items worth calling out - let me know if you don't like the sound of them. Apart from that, I'll proceed as discussed - setup the schema, move the users to the database, add a user UI control, add a logout function, adding (very basic) user management pages, and fixing the implementation of read-only vs edit mode in the UI code.

@matthewberryman
Copy link
Contributor Author

Sounds good—I had envisaged two roles editor and admin (and users can be assigned one or the other or both, hence two columns).

@benatwork99
Copy link
Contributor

I think this has now been all handled or spawned separate tasks, see:

smart-facility/cognicity-rem-server#7
smart-facility/cognicity-rem-server#1
smart-facility/cognicity-rem-server#23
smart-facility/cognicity-rem-server#24

Closing this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants