-
Notifications
You must be signed in to change notification settings - Fork 1
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
Copy tags when using content from a V2 Library in a Course #177
Comments
This is blocked until we get some product input from users. |
What would it take to implement:
|
@jmakowski1123 1 and 3 are relatively easy. For 2, I think it would take quite a bit of work - changes both on the backend and all the required UX. I'd probably recommend de-scoping it at first until we get user feedback. |
Got it. What if 2 was limited to "Users can add new tags to library content when it is being used in a course, but those new tags don't sync back to the Library version of the content." Would that be an easier lift? Then the behavior for tags would follow the exact same pattern as the current behavior for changing Library content when it's being used in a course, ie changes can be made locally in the course but don't sync back to the Library. |
@jmakowski1123 Yep, I think that's going to be a lot easier to implement. |
@jmakowski1123 Let us know if you'd like to proceed with estimating & planning that ? |
Yes please! |
@jmakowski1123 I added a fourth criteria as well - hope it makes sense. Let me know if you want any changes.
|
@jmakowski1123 @lizc577 @sdaitzman @marcotuts, I have one question here
If the tag is copied, then I will hide the delete icon button, but is there a place where the user is informed about this? That these types of tags cannot be deleted? |
I don't think we have a UI solution for this. We need to get there - if this doesn't make the MVP with a message/icon/notification, it's not a blocker. |
@jmakowski1123 @sdaitzman This hasn't merged yet, but you can preview it on the sandbox. Please test it and let us know if there's any issues. |
@jmakowski1123 @sdaitzman You can follow these steps: Create a new library with components. Add tags to these components.
Note: Tags aren't part of the draft-publish cycle, so you have to make changes to the block and publish them before the tags will be synced. |
@jmakowski1123 @lizc577 @sdaitzman @marcotuts friendly reminder that this is ready for AC testing on the sandbox |
"As a content author, when I import library content into a Course, I expect the tags to be copied over as well"
"As a content author, when I import library content into a course, I can add my own tags to that content."
Acceptance Criteria
For this discovery ticket, plan and estimate the implementation of the following:
This only applies to v2 Libraries ("Libraries Relaunch") based on Learning Core / Blockstore - not modulestore.
Implementation will be done in #193
Older Notes
This issue was split off from #165 since it is large enough and a lot of different cases that need to be thought through.
Related Slack thread: https://openedx.slack.com/archives/C03T0RZEASG/p1705345057688579
Main point to address:
If a problem is tagged "easy" in the library, and the tag comes with it when it's used in a course, can authors add additional tags? change or delete those tags from the library? And if so, do we need to keep their changes even after we "update" the content to a newer version (with updated tags) from the library?
i.e.:
Notes:
From a technical perspective, several simple options exist:
The text was updated successfully, but these errors were encountered: