Skip to content

Removes abilities from admins.#9155

Merged
ashercodeorg merged 6 commits into
stagingfrom
restrictAdmins
Jul 6, 2016
Merged

Removes abilities from admins.#9155
ashercodeorg merged 6 commits into
stagingfrom
restrictAdmins

Conversation

@ashercodeorg

Copy link
Copy Markdown
Contributor

Also adds the Game ability to users with UserPermission::LEVELBUILDER.

@ashercodeorg

Copy link
Copy Markdown
Contributor Author

Thanks for your help @aoby with that last commit!

Comment thread dashboard/app/models/ability.rb Outdated
Level,
Script,
ScriptLevel
ScriptLevel,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: trailing , not needed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done (remnant of debugging).

@aoby

aoby commented Jun 28, 2016

Copy link
Copy Markdown
Contributor

Please give a little more detail about the deferred work in that TODO comment.

Otherwise, LGTM

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants