Skip to content

Comments

WFPREV-839 : Create migration to fix typo (Omenica -> Omineca)#1067

Merged
sukhpalp merged 6 commits intomainfrom
wfprev-839
Jan 9, 2026
Merged

WFPREV-839 : Create migration to fix typo (Omenica -> Omineca)#1067
sukhpalp merged 6 commits intomainfrom
wfprev-839

Conversation

@sukhpalp
Copy link
Collaborator

@sukhpalp sukhpalp commented Jan 6, 2026

No description provided.

@sukhpalp sukhpalp requested review from hsaldyga and kcguise January 6, 2026 01:08
dhlevi
dhlevi previously approved these changes Jan 7, 2026
@@ -0,0 +1,3 @@
UPDATE wfprev.forest_org_unit
SET org_unit_name = REPLACE(org_unit_name, 'Omenica', 'Omineca')
WHERE org_unit_name LIKE '%Omenica%';
Copy link
Contributor

Choose a reason for hiding this comment

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

When making a specific change like spelling, I prefer to also use an explicit clause so we only update the code(s) that we're intending to change without potentially changing stuff we didn't mean to with a LIKE. No need to change it here for this case, just a general bit of advice

@sukhpalp sukhpalp merged commit 9e55004 into main Jan 9, 2026
4 of 5 checks passed
@sukhpalp sukhpalp deleted the wfprev-839 branch January 9, 2026 00:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants