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

Add validation check for mismatched Content Group settings in Components #14

Closed
Tracked by #1
ormsbee opened this issue Nov 15, 2022 · 2 comments
Closed
Tracked by #1

Comments

@ormsbee
Copy link

ormsbee commented Nov 15, 2022

No description provided.

@ormsbee ormsbee changed the title Add validation check for mismatched Content Group settings Add validation check for mismatched Content Group settings in Components Nov 15, 2022
@bradenmacdonald
Copy link
Contributor

bradenmacdonald commented Jun 1, 2023

@jmakowski1123 @ormsbee @ali-hugo

I think this already exists in the platform.

Here's what happens now. Note that this XBlock is restricted to a particular content group:

Screenshot 2023-05-31 at 6 12 57 PM

If I copy it and paste into the same course, the content group is preserved as you'd expect.

If I paste it into a different course, I get this warning:

Screenshot 2023-05-31 at 6 14 19 PM

This seems to be compatible with the technical approach doc which says: "For the initial MVP, allow incompatible items to be pasted, and surface the errors in the Studio UI."

So: is this behavior sufficient, or do we want something different?

@ormsbee
Copy link
Author

ormsbee commented Jun 1, 2023

@bradenmacdonald: yeah, I think the existing UI is sufficient–I just wasn't aware of it when I wrote this. I'm closing this for now–anyone should feel free to re-open it if you think it needs further discussion.

@ormsbee ormsbee closed this as completed Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

2 participants