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

[BUG][LUCY-273]** Update init migration to add major_cities and seed data #1159

Merged
merged 1 commit into from
Dec 15, 2023
Merged

[BUG][LUCY-273]** Update init migration to add major_cities and seed data #1159

merged 1 commit into from
Dec 15, 2023

Conversation

davidclaveau
Copy link
Contributor

Pull Request Standards

  • The title of the PR is accurate
  • The title includes the type of change [HOTFIX, FEATURE, etc]
  • The PR title includes the ticket number in format of [LUCY-###]
  • Documentation is updated to reflect change

Description

This PR includes the following proposed change(s):

  • Update the initialization of the LUCY database locally to include major_cities table and seed data
  • Add SQL INSERTs to load city data
  • Issue with invasivebc schema locally, remove from the original sql file

@davidclaveau davidclaveau linked an issue Dec 15, 2023 that may be closed by this pull request
Fix initialization of database to include Major Cities bcgov/invasivesBC-mussels-iOS#273
Closed
LocalNewsTV
LocalNewsTV approved these changes Dec 15, 2023
View reviewed changes
Copy link
Collaborator

@LocalNewsTV LocalNewsTV left a comment

Choose a reason for hiding this comment

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

🚀

@davidclaveau davidclaveau merged commit b3ff4a8 into dev Dec 15, 2023
9 checks passed
@davidclaveau davidclaveau deleted the bug/add-major-cities-table branch December 15, 2023 17:43
@davidclaveau davidclaveau mentioned this pull request Mar 7, 2024
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LocalNewsTV LocalNewsTV LocalNewsTV approved these changes

Assignees
No one assigned
Labels
Backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix initialization of database to include Major Cities
2 participants
@davidclaveau @LocalNewsTV