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

Hide / error on Libraries v2 pages if !librariesV2Enabled #1448

Open
pomegranited opened this issue Oct 30, 2024 · 4 comments
Open

Hide / error on Libraries v2 pages if !librariesV2Enabled #1448

pomegranited opened this issue Oct 30, 2024 · 4 comments
Assignees
Labels
bug Report of or fix for something that isn't working as intended

Comments

@pomegranited
Copy link
Contributor

Accessing the Library Home page (e.g. https://app.tagging-preview.staging.do.opencraft.hosting/course-authoring/library/lib:OpenCraft:TL) should give an error if the "disable v2 libraries" waffle flag is set.

It's totally fine if this error happens after the initial page load - since having v2 libraries enabled is the default we don't want to block loading until the waffle flag is checked.

@pomegranited pomegranited moved this to Backlog in Libraries Overhaul Oct 30, 2024
@pomegranited pomegranited added the bug Report of or fix for something that isn't working as intended label Oct 30, 2024
@bradenmacdonald bradenmacdonald moved this from Backlog to In Progress in Libraries Overhaul Oct 30, 2024
@pomegranited pomegranited moved this from In Progress to Ready for AC testing in Libraries Overhaul Nov 7, 2024
@pomegranited
Copy link
Contributor Author

@jmakowski1123 @lizc577 @sdaitzman @marcotuts This is ready for AC testing on the sandbox

@ChrisChV
Copy link
Contributor

@jmakowski1123 @lizc577 @sdaitzman @marcotuts Friendly reminder that this is ready for AC testing on the sandbox

@jmakowski1123
Copy link

I'm not sure how to test this, since V2 is enabled on the sandbox?

@pomegranited
Copy link
Contributor Author

@jmakowski1123 You can disable V2 on the sandbox using Django Admin:

  1. In Django Admin, enable the contentstore.new_studio_mfe.disable_new_libraries flag to disable Libraries V2.
  2. Load a library's home page in the Authoring MFE.
  3. Verify that the error message is shown instead of the library content.
  4. In Django Admin, disable the contentstore.new_studio_mfe.disable_new_libraries flag to enable Libraries V2.
  5. Refresh the library's home page
  6. Verify that the error message is no longer shown.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Report of or fix for something that isn't working as intended
Projects
Status: Ready for AC testing
Development

No branches or pull requests

3 participants