Skip to content
This repository has been archived by the owner on Jan 4, 2023. It is now read-only.

[2.0] Replace permissions.reset_permissions() clutter by merging config permissions and default permissions at runtime #542

Closed
jlu5 opened this issue Oct 15, 2017 · 0 comments
Closed

[2.0] Replace permissions.reset_permissions() clutter by merging config permissions and default permissions at runtime #542

jlu5 opened this issue Oct 15, 2017 · 0 comments
Labels
release blocker issues/PRs blocking the next release type: bug involves a bug or bug fix type: cleanup issues involving cleaning up (old) code type: enhancement improvements to existing features
Milestone
v2.0

Comments

@jlu5
Copy link
Collaborator

jlu5 commented Oct 15, 2017 •
edited

Default permissions are currently broken in 2.x due to 3b6c1e5, as they are applied before plugins are loaded.
@jlu5 jlu5 added type: cleanup issues involving cleaning up (old) code type: enhancement improvements to existing features labels Oct 15, 2017
@jlu5 jlu5 added this to the v2.0 milestone Oct 15, 2017
@jlu5 jlu5 added type: bug involves a bug or bug fix release blocker issues/PRs blocking the next release labels Oct 15, 2017
jlu5 added a commit that referenced this issue Oct 22, 2017
@jlu5
Move permission enumeration to runtime, fix default perms not applyin…
d0dff2c 
…g at startup

Closes #542.
@jlu5 jlu5 closed this as completed Oct 22, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
release blocker issues/PRs blocking the next release type: bug involves a bug or bug fix type: cleanup issues involving cleaning up (old) code type: enhancement improvements to existing features
Projects
None yet
Milestone
v2.0
Development

No branches or pull requests
1 participant
@jlu5