Skip to content

BAH-2122 | Adds liquibase migration to add default data into tables#27

Open
mohan-13 wants to merge 2 commits into
masterfrom
BAH-2122
Open

BAH-2122 | Adds liquibase migration to add default data into tables#27
mohan-13 wants to merge 2 commits into
masterfrom
BAH-2122

Conversation

@mohan-13

@mohan-13 mohan-13 commented Aug 4, 2022

Copy link
Copy Markdown
Member

This PR adds a liquibase migration that adds default data mapping to modality and order_type tables.

@mohan-13 mohan-13 requested a review from angshu August 4, 2022 05:29
<column name="comment" type="varchar(255)"/>
</addColumn>
</changeSet>
<changeSet id="PACS-20220804" author="Mohankumar">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

please add the column names

INSERT INTO table_name (column1, column2 ...) VALUES (value1, value2 ..)

<column name="comment" type="varchar(255)"/>
</addColumn>
</changeSet>
<changeSet id="PACS-20220804" author="Mohankumar">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

also, will this work on on-premise install? the node name would be localhost or ipaddress, right?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No Angshu. In an on-prem installation, this needs to be updated either with localhost or the ip of where DCM4CHEE (PACS) is running.

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