Skip to content

DB Schema upgrade - #5388

Draft
eddherrera wants to merge 3 commits into
bcgov:devfrom
eddherrera:scaffold/s123
Draft

DB Schema upgrade#5388
eddherrera wants to merge 3 commits into
bcgov:devfrom
eddherrera:scaffold/s123

Conversation

@eddherrera

Copy link
Copy Markdown
Collaborator

Stories:
PSP-11380 : Owner representative comment field

Schema changes:

  • Alter table [PIMS_ACQUISITION_FILE] - new columN [OWNER_REP_COMMENT]
  • Alter table [PIMS_ACQUISITION_FILE_HIST] - new columN [OWNER_REP_COMMENT]

@eddherrera eddherrera self-assigned this Aug 13, 2026
@eddherrera eddherrera added db-migration db-scaffold Updated db models using scaffold labels Aug 13, 2026
@eddherrera

Copy link
Copy Markdown
Collaborator Author

DB Changes in preparation for sprint 127

@github-actions

Copy link
Copy Markdown
Contributor

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/5388

PRINT N'Migrate the PIMS_INTEREST_HOLDER comment data to the PIMS_ACQUISITION_FILE table'
GO

UPDATE acq

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Reviewers: Please pay attention to this SQL DML statement. Review thoroughly.

# Conflicts:
#	source/database/mssql/scripts/dbscripts/PSP_PIMS_LATEST/Build/020_DML_PIMS_STATIC_VARIABLE.sql
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
80.2% Duplication on New Code (required ≤ 3%)
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@github-actions

Copy link
Copy Markdown
Contributor

See CodeCov Report Here: https://app.codecov.io/github/bcgov/psp/pull/5388

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

Labels

db-migration db-scaffold Updated db models using scaffold

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant