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

chore: Add django admin for RestrictedCourseMetadata #970

Merged
merged 1 commit into from
Oct 11, 2024

Conversation

pwnage101
Copy link
Contributor

@pwnage101 pwnage101 commented Oct 11, 2024

ENT-9569

Also, this should be a usability/QoL improvement for catalog, catalog query, and content metadata fields in other admin pages.

Screenshots

List RestrictedCourseMetadata:

  • Note: first column links to record, 2nd and 3rd columns link to other objects.

Screenshot 2024-10-10 at 6 03 48 PM

Edit a restricted course (RestrictedCourseMetadata):
Screenshot 2024-10-10 at 6 02 33 PM

Edit a restricted run (ContentMetadata):
Screenshot 2024-10-10 at 6 04 48 PM

Edit a course with lots of catalog/catalogquery relationships (ContentMetadata):

  • Note that "Catalog Queries" and "Enterprise Catalogs" are clean lists now (instead of multiple objects on one line), and link back to the object.

Screenshot 2024-10-10 at 6 03 31 PM

@pwnage101 pwnage101 force-pushed the pwnage101/ENT-9569-2 branch from a3008dd to 5a68c05 Compare October 11, 2024 00:53
@pwnage101 pwnage101 force-pushed the pwnage101/ENT-9569-2 branch from 5a68c05 to 51b6531 Compare October 11, 2024 01:02
@pwnage101
Copy link
Contributor Author

If I had more time tonight, I'd make a course in ContentMetadata list out all its restricted versions.

@iloveagent57 iloveagent57 merged commit 0f6874a into master Oct 11, 2024
4 checks passed
@iloveagent57 iloveagent57 deleted the pwnage101/ENT-9569-2 branch October 11, 2024 16:45
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.

2 participants