Skip to content

fix: update modal title to reflect edit vs add signer action#6157

Merged
vitormattos merged 1 commit intomainfrom
fix/edit-signer-modal-title
Dec 12, 2025
Merged

fix: update modal title to reflect edit vs add signer action#6157
vitormattos merged 1 commit intomainfrom
fix/edit-signer-modal-title

Conversation

@vitormattos
Copy link
Copy Markdown
Member

The modal dialog was displaying 'Add new signer' title when clicking an existing signer to edit, which was confusing. Added a computed property 'modalTitle' that dynamically returns 'Edit signer' when editing an existing signer or 'Add new signer' when adding a new one.

The logic checks if signerToEdit object has properties to determine the correct context.

The modal dialog was displaying 'Add new signer' title when clicking
an existing signer to edit, which was confusing. Added a computed
property 'modalTitle' that dynamically returns 'Edit signer' when
editing an existing signer or 'Add new signer' when adding a new one.

The logic checks if signerToEdit object has properties to determine
the correct context.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos vitormattos added this to the Next Major (33) milestone Dec 12, 2025
@vitormattos vitormattos self-assigned this Dec 12, 2025
@github-project-automation github-project-automation Bot moved this to 0. Needs triage in Roadmap Dec 12, 2025
@vitormattos
Copy link
Copy Markdown
Member Author

/backport to stable32

@vitormattos
Copy link
Copy Markdown
Member Author

/backport to stable31

@vitormattos vitormattos merged commit 4535979 into main Dec 12, 2025
57 checks passed
@vitormattos vitormattos deleted the fix/edit-signer-modal-title branch December 12, 2025 16:40
@github-project-automation github-project-automation Bot moved this from 0. Needs triage to 4. to release in Roadmap Dec 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant