Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

❗ Confusing Stake Error During Proposal Submission with Unregistered Reward Address #2536

Open
kneerose opened this issue Dec 17, 2024 · 2 comments

Comments

@kneerose
Copy link
Contributor

Area

Proposal Pillar

Domain

dev-sanchogov.tools

Which wallet were you using?

No response

Context

  • A treasury proposal submission fails if the reward address used during proposal creation is linked to an unregistered stake.
  • Although the wallet stake is registered, the system currently allows proposal creation without validation for the reward address.
  • When submitting the proposal, the error message indicates that the wallet's stake is not registered, which can be confusing.
  • The proposal creator may be unaware that the issue lies with the unregistered reward address rather than their registered wallet stake.
  • It would be helpful to notify the creator during the proposal creation phase if the reward address's stake is not registered.

Steps to reproduce

  1. Connect to the wallet.
  2. Create a treasury proposal with a reward address whose stake is not registered (e.g., stake1up0nm089rryg9w8kv0x60xf0u6e44zy92suaxagwjc0ewqgn273q3).
  3. Submit the proposal to the governance action page.
  4. Observe the error in the console.

Actual behavior

  • The system allows proposal creation even if the reward address's stake is unregistered. However, when submitting the proposal, the console displays an error indicating that the wallet stake is not registered, without clarifying that the reward address is the issue.

image

Expected behavior

  • If the reward address's stake is not registered, this should be identified and communicated during the proposal creation phase. The system should notify the proposal creator that the reward address's stake is unregistered, avoiding confusion with the wallet's registered stake.
@MSzalowski
Copy link
Contributor

CC @nebojsact @spannercode

That functionality is rather a feature request as we don't have such logic implemented on GovTool yet.

We would need to provide a stake key registration check (by some EP on PDF or GovTool side). With such an EP, I think we should check that on a form level and not submit the transaction with the unregistered stake key.

Also as @spannercode described on a call that might be another issue with not catching the failed transaction submission which resulted in not displaying an error modal on submission.

@MSzalowski
Copy link
Contributor

GET /ada-holder/get-voting-power/{stakeKey} Could be used so far, as this EP checks existence of a stake key except providing the Ada holder voting power.

@MSzalowski MSzalowski removed their assignment Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Status: To do
Development

No branches or pull requests

5 participants
@MSzalowski @nebojsact @kneerose @bosko-m and others