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

Permissions API support concept 2 #1864

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Permissions API support concept 2 #1864

wants to merge 4 commits into from
+42 −19

Commits on Jan 19, 2024

  1. Permissions API support concept 2

    @altrisi
    altrisi committed Jan 19, 2024
    Configuration menu
    Copy the full SHA
    60f54ef View commit details
    Browse the repository at this point in the history

  2. Add note about permissions support being available 

    TBD: Not convinced to add it here, given extension commands may not support it
    @altrisi
    altrisi committed Jan 19, 2024
    Configuration menu
    Copy the full SHA
    2d20992 View commit details
    Browse the repository at this point in the history

  3. Use permission-node-capable method in command permission checking 

    This will have to be pending the Scarpet command refactor thing to not
have conflicts
    @altrisi
    altrisi committed Jan 19, 2024
    Configuration menu
    Copy the full SHA
    d760f97 View commit details
    Browse the repository at this point in the history

  4. Hook it to SettingsManager too 

    Hooking to Scarpet app commands is a bit more complex
    @altrisi
    altrisi committed Jan 19, 2024
    Configuration menu
    Copy the full SHA
    c44f698 View commit details
    Browse the repository at this point in the history