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.

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

Institutional managers allowed box #174

Open
JohnMcLeod13 opened this issue May 27, 2024 · 3 comments
Open

Institutional managers allowed box #174

JohnMcLeod13 opened this issue May 27, 2024 · 3 comments

Comments

@JohnMcLeod13
Copy link

I would expect that you can not enter an institutional manager name unless you check the box "Institutional Managers Allowed." Currently on the shared staging network, you can add the a user as the institutional manager without the allowed box checked.

@greatislander
Copy link
Contributor

See also: #129 (comment)

Add/Edit Institution: The “Institution Managers Allowed” checkbox does not have any perceivable relationship to or impact on the “Institutional Managers” field. If it allows or disallows the use of the “Institutional Managers” field, the field should be hidden or shown as appropriate (or disabled or enabled as appropriate).

@SteelWagstaff
Copy link
Member

I believe this was intentional -- the idea being that super admins (i.e. Pressbooks staff) could add institutional managers no matter what this setting was? Network managers should not be able to add institutional managers unless this box (or the buy-in box) are checked.

@greatislander
Copy link
Contributor

I believe this was intentional -- the idea being that super admins (i.e. Pressbooks staff) could add institutional managers no matter what this setting was? Network managers should not be able to add institutional managers unless this box (or the buy-in box) are checked.

It needs a clearer label, then.

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

No branches or pull requests

3 participants