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 model specs #82

Closed
2 of 5 tasks
javorszky opened this issue May 27, 2013 · 2 comments · Fixed by #131
Closed
2 of 5 tasks

User model specs #82

javorszky opened this issue May 27, 2013 · 2 comments · Fixed by #131
Assignees
Milestone

Comments

@javorszky
Copy link
Contributor

Right, so here's the idea:

  • first user ever is admin, auto-login
  • subsequent users be not auto-logged in, but somehow be flagged in the admin area that they've requested access, and had to be approved? (discuss)
  • there should be check for duplicate email addresses and usernames
  • fixtures updated with username: john@onolan.org, pass: Wh0YouG0nnaCall?
  • email address validation

Questions

  • When should we send an email?
    • password reset
    • verification
    • approval
  • setting to allow signup for anyone with a specific role (how's that ACL coming along?)?
    • would this be a white-list kind of thing
    • a simple dropdown with [no, reader, contributor, editor, user, admin]
    • a checkbox and a dropdown... ? )

Anything else I missed, let me know, otherwise I'm working on stuff that I think are core at this stage, which are:

@ghost ghost assigned javorszky May 27, 2013
@ErisDS
Copy link
Member

ErisDS commented May 27, 2013

A couple of super quick things:

  • registration very much relies on / is connected to ACL - Jacob has taken that on and I'm working on specing out some of the perms / roles etc - see ACL
  • for standalone installs,first-user (admin) registration needs to happen as part of the install process
  • pretty much everything else you've said is spot on

@javorszky
Copy link
Contributor Author

Because the duplicate email address validation is broken. Yay...

morficus pushed a commit to morficus/Ghost that referenced this issue Sep 4, 2014
User authentication / registration, fixture fix closes TryGhost#82
morficus pushed a commit to morficus/Ghost that referenced this issue Sep 4, 2014
There was a return missing. This closes TryGhost#82 again.
@renovate renovate bot mentioned this issue Oct 25, 2022
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants