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.

Sign up for GitHub

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

Jump to bottom

Use the new transaction verification results #35

Open
abizjak opened this issue Jan 31, 2022 · 0 comments
Open

Use the new transaction verification results #35

abizjak opened this issue Jan 31, 2022 · 0 comments
Labels
[Type] Change Request Some visible functionality should be change. [Type] Task An additional feature or improvement.

Comments

@abizjak
Copy link
Contributor

abizjak commented Jan 31, 2022

Task description

When submitting transactions the proxy currently returns an error if the transaction is duplicate.

Since the relevant endpoints are PUT, the correct response would be to accept. This was not possible before since we did not get the information on whether the transaction was duplicate or not, just that it was not accepted.

With changes in Concordium/concordium-node#217 we do have this information, which we should use.
@abizjak abizjak added [Type] Change Request Some visible functionality should be change. [Type] Task An additional feature or improvement. labels Jan 31, 2022
@MilkywayPirate MilkywayPirate mentioned this issue Feb 3, 2022
Closed
5 tasks
@MilkywayPirate MilkywayPirate removed their assignment Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
[Type] Change Request Some visible functionality should be change. [Type] Task An additional feature or improvement.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@abizjak @MilkywayPirate