Skip to content

Conversation

@sarina sarina force-pushed the sarina/ulmo-authz-console branch 3 times, most recently from 3723365 to d21e7d6 Compare December 15, 2025 17:11
@sarina sarina marked this pull request as ready for review December 15, 2025 17:12
Copy link
Contributor

@MaferMazu MaferMazu left a comment

Choose a reason for hiding this comment

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

Thanks for this PR @sarina and @gviedma-aulasneo

I still need to review the ulmo_rp and the how-to of adding a user, I'll do tomorrow, but I am sharing what I have regarding the rest.

Note: probably in the future we want to unify the roles and permissions page here or in authz, but it would be great to have only one page so we don't repeat ourselves and make it easier to maintain when we add more default roles and permissions.

Copy link
Contributor

@MaferMazu MaferMazu left a comment

Choose a reason for hiding this comment

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

I just finished reviewing the ulmo_rp and the how-to to add roles.

In general, it's looking well. I only added some suggestions.

Thanks again for this work.

@sarina sarina force-pushed the sarina/ulmo-authz-console branch from 39f7e80 to 6c37706 Compare December 19, 2025 21:20
@sarina
Copy link
Contributor Author

sarina commented Dec 19, 2025

@MaferMazu @mariajgrimaldi I removed the Library R&P pages and did cross-references to the authz docs in a few places. I did wholesale copy-paste the table from https://docs.openedx.org/projects/openedx-authz/en/latest/concepts/core_roles_and_permissions/content_library_roles.html#roles-and-permissions-summary-table into the Ulmo release note because I couldn't embed it unfortunately, but since it's point-in-time documentation I think it's okay. Can I go ahead and merge this?

Copy link
Contributor

@MaferMazu MaferMazu left a comment

Choose a reason for hiding this comment

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

✅ Everything looks good to me. (only a minor comment no blocker)

Thanks @sarina

before it can be used in a Course.

A course author must have at least Read-only permissions to reuse the content.
A course author must have at least the **Library User** role to reuse content.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
A course author must have at least the **Library User** role to reuse content.
A course author must have at least the **Library User** role to reuse content, *or*,
the library should be published with the **Allow Public Read** permission.

Copy link
Contributor

Choose a reason for hiding this comment

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

(no blocker)

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

Successfully merging this pull request may close these issues.

Update content libraries roles and permissions documentation

6 participants