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

Improve SDK Behavior for Duplicate Statement Anchoring #177

Closed
adi-a11y opened this issue Feb 28, 2024 · 1 comment
Closed

Improve SDK Behavior for Duplicate Statement Anchoring #177

adi-a11y opened this issue Feb 28, 2024 · 1 comment

Comments

@adi-a11y
Copy link
Contributor

Description:
When attempting to anchor a statement that is already present on the chain using the SDK, the current behaviour is confusing. The SDK returns the identifier without errors and no events are observed on the chain. It becomes perplexing because, while the terminal indicates that the dispatch call has been executed, there are no corresponding events observable on the chain. Consequently, one is left pondering why no error is displayed or why the event is absent on the chain, despite the SDK indicating successful execution without errors.

Proposed Fix:
If the statement is already anchored on the chain, the SDK should throw an error indicating "The statement already exists" and should also return the identifier for reference.

@vatsa287
Copy link
Member

vatsa287 commented Aug 9, 2024

Closing as completed with #178.

@vatsa287 vatsa287 closed this as completed Aug 9, 2024
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

No branches or pull requests

2 participants