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

Truncate NVARCHAR(max) Columns Prior to Alter Up #4224

Merged
merged 7 commits into from
Jul 24, 2024
Merged

Truncate NVARCHAR(max) Columns Prior to Alter Up #4224

merged 7 commits into from
Jul 24, 2024

Conversation

dfilteau
Copy link
Contributor

No description provided.

@dfilteau
IS-85.00 Database Schema
3ea9cfb 
PSP_PIMS | Development Sprint: S85.00 | Design Sprint: 84 | Date: 2024-JuL-11
- Added tables:
  - PIMS_LEASE_LEASE_PURPOSE
  - PIMS_LEASE_LEASE_PURPOSE_HIST
- Altered tables:
  - PIMS_ACCESS_REQUEST
  - PIMS_ACCESS_REQUEST_HIST
  - PIMS_ACQUISITION_FILE_FORM
  - PIMS_ACQUISITION_FILE_FORM_HIST
  - PIMS_LEASE
  - PIMS_LEASE_HIST
  - PIMS_PROPERTY
  - PIMS_PROPERTY_HIST
  - PIMS_PROPERTY_RESEARCH_FILE
  - PIMS_PROPERTY_RESEARCH_FILE_HIST
  - PIMS_RESEARCH_FILE
  - PIMS_RESEARCH_FILE_HIST
  - PIMSX_TableDefinitions
- Dropped tables:
  - PIMS_LEASE_CATEGORY_TYPE
- Altered seed scripts:
  - 057_DML_PIMS_LEASE_PROGRAM_TYPE.sql
  - 136_DML_PIMS_PROP_MGMT_ACTIVITY_SUBTYPE.sql
- Dropped seed scripts:
  - 055_DML_PIMS_LEASE_CATEGORY_TYPE.sql
- Altered test scripts:
  - 004_DML_LEASE_LOAD.sql
- Requires additional metadata to meet standards
@dfilteau
Updated Alter Down and Added View Drop/Create
ac36167
@dfilteau
Updated Alter Up
df1ce92
@dfilteau
Merge branch 'bcgov:dev' into IS-85.00
b6d9e04
@dfilteau
Added 001_Reset_NVARCHAR_Max_Columns_Alter_Down.sql to Alter Up
0d320b3
@dfilteau
Merge branch 'IS-85.00' of https://github.com/dfilteau/PSP into IS-85.00
edafae8
@dfilteau
Truncate NVARCHAR Max Columns
5b68aed
@github-actions GitHub Actions
Copy link
Contributor

✅ No secrets were detected in the code.

asanchezr
asanchezr approved these changes Jul 24, 2024
View reviewed changes
Copy link
Collaborator

@asanchezr asanchezr left a comment

Choose a reason for hiding this comment

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

Looks good to me but need Manuel to review in case I missed something

@FuriousLlama FuriousLlama self-assigned this Jul 24, 2024
@FuriousLlama FuriousLlama merged commit 31597d7 into bcgov:dev Jul 24, 2024
7 checks passed
@dfilteau dfilteau deleted the IS-85.00 branch July 31, 2024 17:14
This was referenced Aug 19, 2024
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@asanchezr asanchezr asanchezr approved these changes

@FuriousLlama FuriousLlama FuriousLlama approved these changes

Assignees

@FuriousLlama FuriousLlama

Labels
db-migration db-migration-correction
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dfilteau @FuriousLlama @asanchezr