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

293 rename hame to arho #337

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

293 rename hame to arho #337

wants to merge 13 commits into from

Conversation

Mtk112
Copy link

@Mtk112 Mtk112 commented Sep 12, 2024 •
edited
Loading

#293

  • Renames hame-schema and references to arho.
  • Replaced previous migrations with two new migrations.
  • Modifies tests to reflect changes.
  • Updated database documentation

@Mtk112 Mtk112 linked an issue Sep 12, 2024 that may be closed by this pull request
Uudelleennimeä projekti HAME-Ryhti -> QGIS-Ryhti #293
Open
@Mtk112 Mtk112 self-assigned this Sep 12, 2024
@Mtk112 Mtk112 marked this pull request as ready for review September 17, 2024 10:45
Rikuoja
Rikuoja requested changes Sep 26, 2024
View reviewed changes
Copy link
Contributor

@Rikuoja Rikuoja 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, but let's first rebase this on current main (fix merge conflicts) and then second, I should edit the PR to disable CI/CD. So don't merge this after rebase, I will do an extra commit on top of that.

README.md Outdated Show resolved Hide resolved
@Mtk112
Rename hame to arho
85ccae8 
Rename hame-schema to arho.
Recreate triggers. Change hame refrences.
@Mtk112
Update database documentation
cb0a2cd 
Regenerated database documentation.
@Mtk112
Rename tfvars.jsons
94f5b13
@Mtk112
Updated .tbls.yml and documentation
295c14b
@Mtk112 Mtk112 linked an issue Sep 30, 2024 that may be closed by this pull request
Terraform-konffin hame-viittaukset arhoiksi #350
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rikuoja Rikuoja Rikuoja requested changes

Requested changes must be addressed to merge this pull request.

Assignees

@Mtk112 Mtk112

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Terraform-konffin hame-viittaukset arhoiksi Uudelleennimeä projekti HAME-Ryhti -> QGIS-Ryhti
2 participants
@Mtk112 @Rikuoja