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.

Sign up for GitHub

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

Jump to bottom

Remove studio_post_duplicate XBlock hook method #35192

Open
Tracked by #33640
kdmccormick opened this issue Jul 26, 2024 · 1 comment
Open
Tracked by #33640

Remove studio_post_duplicate XBlock hook method #35192

kdmccormick opened this issue Jul 26, 2024 · 1 comment
Assignees
@Ian2012
Labels
code health Proactive technical investment via refactorings, removals, etc.

Comments

@kdmccormick
Copy link
Member

kdmccormick commented Jul 26, 2024 •
edited
Loading

This ad-hoc hook was added to account for edge-cases when duplicating or copy-pasting LibraryContentBlocks. It will no longer be necessary once we remove V1 content libraries.
@kdmccormick kdmccormick added the content libraries misc Libraries Overhaul tech work not captured in the stories label Jul 26, 2024
@kdmccormick kdmccormick mentioned this issue Jul 26, 2024
Closed
@kdmccormick kdmccormick added the code health Proactive technical investment via refactorings, removals, etc. label Jul 26, 2024
@kdmccormick kdmccormick removed the content libraries misc Libraries Overhaul tech work not captured in the stories label Aug 26, 2024
@kdmccormick kdmccormick mentioned this issue Aug 26, 2024
Open
@Ian2012 Ian2012 self-assigned this Oct 24, 2024
@kdmccormick
Copy link
Member Author

@Ian2012 LegacyLibraryContentBlock uses this hook, so we will have to keep it until we migrate V1 libs to V2 libs and migrate all LegacyLibraryContentBlock usages to ItemBankBlock usages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ian2012 Ian2012

Labels
code health Proactive technical investment via refactorings, removals, etc.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kdmccormick @Ian2012