Skip to content

LT-22519: ensure variant form title uses the variant version#898

Merged
AndyBlack merged 1 commit into
mainfrom
LT22519
May 19, 2026
Merged

LT-22519: ensure variant form title uses the variant version#898
AndyBlack merged 1 commit into
mainfrom
LT22519

Conversation

@AndyBlack
Copy link
Copy Markdown
Collaborator

@AndyBlack AndyBlack commented May 19, 2026

This fixes https://jira.sil.org/browse/LT-22519 (Dialog label changes incorrectly from "Variant Generator" to "Allomorph Generator" after clicking Save Changes).

The VariantGenForm subclasses AlloGenFormBase which has a protected variable called formTitle. This gets initialized to "Allomorph Generator". The fix is to ensure it gets set to "Variant Generator" in the VariantGenForm.


This change is Reviewable

Change-Id: Id183934d7b7dcdeb85ff3d97ef7668b6c97d4f8b
@github-actions
Copy link
Copy Markdown

NUnit Tests

    1 files  ±0      1 suites  ±0   12m 3s ⏱️ +50s
4 205 tests ±0  4 134 ✅ ±0  71 💤 ±0  0 ❌ ±0 
4 214 runs  ±0  4 143 ✅ ±0  71 💤 ±0  0 ❌ ±0 

Results for commit 926d836. ± Comparison against base commit d4e947d.

Copy link
Copy Markdown
Contributor

@jasonleenaylor jasonleenaylor left a comment

Choose a reason for hiding this comment

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

:lgtm:

@jasonleenaylor reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on AndyBlack).

@AndyBlack AndyBlack merged commit 272325a into main May 19, 2026
7 checks passed
@AndyBlack AndyBlack deleted the LT22519 branch May 19, 2026 21:19
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.

2 participants