Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

Np 32875 remove migration code #351

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

brinxmat
Copy link
Contributor

@brinxmat brinxmat commented Feb 2, 2023

  • Removes migration code inserted to support:
    • JournalArticle subtypes
    • ChapterArticle subtypes
    • BookMonograph subtype
    • UnconfirmedPublisher

@brinxmat brinxmat changed the base branch from main to NP-15775-migrate-book-monograph-to-discrete-subtypes-based-on-content-type February 2, 2023 21:20
@brinxmat brinxmat changed the base branch from NP-15775-migrate-book-monograph-to-discrete-subtypes-based-on-content-type to NP-15776-migrate-chapter-article-to-discrete-ssubtypes-based-on-content-type February 2, 2023 21:21
@brinxmat brinxmat requested review from ketilaa and torarnet February 2, 2023 21:22
ketilaa
ketilaa previously approved these changes Feb 3, 2023
Copy link
Contributor

@ketilaa ketilaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So you remove the migration code before it has been deployed and migrated?

torarnet
torarnet previously approved these changes Feb 3, 2023
Base automatically changed from NP-15776-migrate-chapter-article-to-discrete-ssubtypes-based-on-content-type to main February 6, 2023 15:22
@brinxmat brinxmat dismissed stale reviews from torarnet and ketilaa via 8eb5ae9 February 6, 2023 15:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants