Forgotten new permissions in the category security tab

Continuing the discussion from Auto-add permissions/unsaved permissions prompt:

I think we need to go a bit further in this.

If you have a pending change (1) and click on save we should stop you and say that you have pending changes to apply, in a “Are you sure” thing.

5 Likes

should we even need to click save? I feel like clicking the add button should be enough?

3 Likes

I like save. It’s an extra step to ensure the permissions are correct. Without it, if I select the wrong group in the dropdown and click the :heavy_plus_sign:, they get access immediately, without an opportunity for me to fix it.

3 Likes

One thing that makes this slightly more complicated is the “Save Category” button applies to content on all tabs/pills right now. The warning should probably apply if you’ve made changes to ANYTHING on any section if you hit the X.

6 Likes

Hey @Osama can you take this one? It should be pretty straightforward but let us know if you encounter any issues.

4 Likes