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

Db contrib/7200 support to removing deprecated measurable categories #7202

Conversation

davidwatkins73
Copy link
Member

Merge in WALTZ/waltz from WALTZ/waltz-kj:db-feature/CTCTOWALTZ-3434-add-deprecated-to-3-taxonomies-in-the-ui to db-feature/7200-support-to-removing-deprecated-measurable-categories

* commit '2b038ba19edf58c6ecc761e8c426196023b434d6':
  updated with reveiw comments
  updated with review comments
  remove depreceted mc
…3 taxonomies in the ui

Merge in WALTZ/waltz from WALTZ/waltz-kj:db-feature/CTCTOWALTZ-3434-add-deprecated-to-3-taxonomies-in-the-ui to db-feature/7200-support-to-removing-deprecated-measurable-categories

* commit '26b9e90b4b533905af6e9ab27adda9b0d3cd21f0':
  updated codebase with new column
  reverted to original codebase
Merge in WALTZ/waltz from WALTZ/waltz-kj:db-feature/CTCTOWALTZ-3434-add-deprecated-to-3-taxonomies-in-the-ui to db-feature/7200-support-to-removing-deprecated-measurable-categories

* commit '23a95f647bd1ca7a6107ac9624a15edde4735889':
  added field into TO_MAPPER
Copy link

linux-foundation-easycla bot commented Jan 24, 2025

CLA Not Signed

value="true"/>
<where>external_id in ('SERVICE','BUSINESS','VRM')</where>
</update>
</changeSet>
Copy link
Member Author

Choose a reason for hiding this comment

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

This changeset belongs in your organisation - not the main repo. We don't want to deprecate for all users across all orgs :)

@@ -24,4 +24,26 @@
author="patidev">
<tagDatabase tag="v1.66"/> <!-- tag the current db version, i.e. the last -->
</changeSet>

<changeSet id="20250123"
Copy link
Member Author

Choose a reason for hiding this comment

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

the change set id's should have the form:

--

where:

  • date is yyyymmdd
  • issue is the waltz issue number (not your orgs), e.g. 7202
  • step is a counter in case a single change is split across multiple steps, e.g. 1, 2

so this changeset id should be 20240123-7202-1

The README.md in this directory (https://github.com/finos/waltz/blob/master/waltz-schema/src/main/resources/liquibase/README.md) has a more detailed explanation

…3 taxonomies in the ui

Merge in WALTZ/waltz from WALTZ/waltz-kj:db-feature/CTCTOWALTZ-3434-add-deprecated-to-3-taxonomies-in-the-ui to db-feature/7200-support-to-removing-deprecated-measurable-categories

* commit '43bf48fcb0e64766ec95326a06d0c165ad52f1f3':
  updating comments
  removed DML
@davidwatkins73
Copy link
Member Author

Thanks for making the changes.

@davidwatkins73 davidwatkins73 merged commit 15c85be into finos:master Jan 24, 2025
0 of 2 checks passed
@davidwatkins73 davidwatkins73 deleted the db-contrib/7200-support-to-removing-deprecated-measurable-categories branch January 24, 2025 11:08
@deutschebank-sync deutschebank-sync restored the db-contrib/7200-support-to-removing-deprecated-measurable-categories branch January 24, 2025 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants