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.

Sign up for GitHub

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

Jump to bottom

Incorrect error when an editor attempts to save the owner #3373

Closed
ErisDS opened this issue Jul 23, 2014 · 0 comments · Fixed by #3380
Closed

Incorrect error when an editor attempts to save the owner #3373

ErisDS opened this issue Jul 23, 2014 · 0 comments · Fixed by #3380
Labels
bug [triage] something behaving unexpectedly
Milestone
0.5 Multi-user

Comments

@ErisDS
Copy link
Member

ErisDS commented Jul 23, 2014

If you log in as an editor, and attempt to save the owner user (which you can currently browse to), this error appears:

Steps to Reproduce

  1. Create a new user
  2. Change the new users role from author to editor
  3. Browse to the owner user (either via direct url, or by going to your profile, hitting the <Users button and selecting the owner)
  4. Attempt to change something simple

Note: despite the error, the changes still get saved because #3096 is not done yet (this is what I was working on when I discovered the issue)

Technical details:
@ErisDS ErisDS added this to the 0.5 Multi-user milestone Jul 23, 2014
@ErisDS ErisDS added the bug label Jul 23, 2014
sebgie added a commit to sebgie/Ghost that referenced this issue Jul 24, 2014
@sebgie
Fix incorrect error
fa054a8 
closes TryGhost#3373
- added check if role is already assigned
- added check for unknown fields to fixDates/fixBools
- permissions are not implemented yet, so everyone is able to edit
owner ;-)
@sebgie sebgie mentioned this issue Jul 24, 2014
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug [triage] something behaving unexpectedly
Projects
None yet
Milestone
0.5 Multi-user
Development

Successfully merging a pull request may close this issue.

Fix incorrect error sebgie/Ghost
1 participant
@ErisDS