Skip to content

fix: add able to sign getter#6705

Merged
vitormattos merged 2 commits intomainfrom
fix/add-able-to-sign-getter
Feb 3, 2026
Merged

fix: add able to sign getter#6705
vitormattos merged 2 commits intomainfrom
fix/add-able-to-sign-getter

Conversation

@vitormattos
Copy link
Copy Markdown
Member

No description provided.

The ableToSign getter was missing from the sign store, causing the
Sign component to fall into the 'Unable to sign' state even when all
conditions were met.

The getter checks:
- Document status is ABLE_TO_SIGN or PARTIAL_SIGNED
- There is a signer with me: true
- Signer status is 1 (ABLE_TO_SIGN)

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Adds comprehensive test coverage for the ableToSign getter including:
- Document status validation
- Signer presence validation
- Signer status validation
- Edge cases (undefined document, empty signers array)

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

/backport to stable33

@vitormattos
Copy link
Copy Markdown
Member Author

/backport to stable32

@vitormattos vitormattos merged commit bd15764 into main Feb 3, 2026
62 checks passed
@vitormattos vitormattos deleted the fix/add-able-to-sign-getter branch February 3, 2026 23:06
@github-project-automation github-project-automation Bot moved this from 0. Needs triage to 4. to release in Roadmap Feb 3, 2026
@vitormattos vitormattos removed this from the Next Major (34) milestone Feb 20, 2026
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