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

B 22122 int #14490

Merged
merged 3 commits into from
Dec 31, 2024
Merged

B 22122 int #14490

merged 3 commits into from
Dec 31, 2024

Conversation

CoryKleinjanCACI
Copy link
Contributor

@CoryKleinjanCACI CoryKleinjanCACI commented Dec 30, 2024

B-22122

Summary

Issue existed where due to existing zip3 conflicts the previous migration file was failing to add all alaska zip3s. This removes existing alaska zip3s, and then adds the entirety of them back to avoid this issue.

How to test

  1. Migrate fresh DB
  2. Validate various rows exists. Issue was found with following sql query:

SELECT * FROM public.zip3_distances
where from_zip3 = '424' AND to_zip3 = '997'
ORDER BY id ASC

@CoryKleinjanCACI CoryKleinjanCACI requested a review from a team as a code owner December 30, 2024 19:54
@CoryKleinjanCACI CoryKleinjanCACI changed the base branch from main to integrationTesting December 30, 2024 19:55
@CoryKleinjanCACI CoryKleinjanCACI requested a review from a team as a code owner December 30, 2024 19:55
@CoryKleinjanCACI CoryKleinjanCACI self-assigned this Dec 30, 2024
@CoryKleinjanCACI CoryKleinjanCACI added the INTEGRATION Slated for Integration Testing label Dec 30, 2024
@CoryKleinjanCACI CoryKleinjanCACI merged commit 84a76a5 into integrationTesting Dec 31, 2024
34 checks passed
@CoryKleinjanCACI CoryKleinjanCACI deleted the B-22122-INT branch December 31, 2024 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INTEGRATION Slated for Integration Testing
Development

Successfully merging this pull request may close these issues.

3 participants