Skip to content

🐛 disallowed inactive users from being made owner#10647

Merged
kevinansfield merged 1 commit into
TryGhost:masterfrom
vikaspotluri123:no-transfer-suspend
Apr 16, 2019
Merged

🐛 disallowed inactive users from being made owner#10647
kevinansfield merged 1 commit into
TryGhost:masterfrom
vikaspotluri123:no-transfer-suspend

Conversation

@vikaspotluri123
Copy link
Copy Markdown
Member

  • closes Should not be possible to transfer ownership to a suspended user #10555
  • Added a check to the user modal that the new owner is active
  • Had to refactor Owner->Author unit test (also renamed it)
    • Based on the first 2 lines, owner->editor change is attempted (hence the rename)
    • Since both stubs return a 'modal' with owner role which means owner->owner change is actually attempted
    • Now that there's a user status check, added the status property to the user receiving owernship

@kevinansfield kevinansfield merged commit c58236e into TryGhost:master Apr 16, 2019
@vikaspotluri123 vikaspotluri123 deleted the no-transfer-suspend branch April 16, 2019 12:53
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 this pull request may close these issues.

Should not be possible to transfer ownership to a suspended user

2 participants