Skip to content

fix: correct Signer component prop in VisibleElements#6151

Merged
vitormattos merged 1 commit intomainfrom
fix/visible-elements-signer-prop
Dec 11, 2025
Merged

fix: correct Signer component prop in VisibleElements#6151
vitormattos merged 1 commit intomainfrom
fix/visible-elements-signer-prop

Conversation

@vitormattos
Copy link
Copy Markdown
Member

The Signer component expects signerIndex prop but VisibleElements was passing current-signer instead. This caused Vue warnings about missing required prop and errors when trying to access signer data.

Changed :current-signer to :signer-index and removed obsolete :signer prop since the component now fetches signer data from the store using the signerIndex.

The Signer component expects signerIndex prop but VisibleElements
was passing current-signer instead. This caused Vue warnings about
missing required prop and errors when trying to access signer data.

Changed :current-signer to :signer-index and removed obsolete
:signer prop since the component now fetches signer data from
the store using the signerIndex.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos vitormattos added this to the Next Major (33) milestone Dec 11, 2025
@vitormattos vitormattos self-assigned this Dec 11, 2025
@github-project-automation github-project-automation Bot moved this to 0. Needs triage in Roadmap Dec 11, 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 4212de8 into main Dec 11, 2025
32 checks passed
@vitormattos vitormattos deleted the fix/visible-elements-signer-prop branch December 11, 2025 23:12
@github-project-automation github-project-automation Bot moved this from 0. Needs triage to 4. to release in Roadmap Dec 11, 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