Skip to content
This repository has been archived by the owner on Jul 27, 2022. It is now read-only.

Revisit tokenaccounts #18

Closed
wants to merge 9 commits into from
Closed

Revisit tokenaccounts #18

wants to merge 9 commits into from

Conversation

BouweCeunen
Copy link
Contributor

No description provided.

@BouweCeunen BouweCeunen mentioned this pull request Dec 11, 2021
Closed
@MaximPiessen MaximPiessen deleted the revisit-tokenaccounts-- branch December 17, 2021 07:21
RuvendeGroote added a commit that referenced this pull request Feb 7, 2022
@RuvendeGroote
fix: adds extra confirming of transactions
8b5a1ae 
Even when waiting for a transaction to hit a certain commitment \
level, we were noticing that calls that happen right after would \
sometimes return stale data.
If a transaction was included in slot #18 some of the calls would \
sometimes return data from that was present at slot #17
@RuvendeGroote RuvendeGroote mentioned this pull request Feb 7, 2022
Closed
RuvendeGroote added a commit that referenced this pull request Feb 7, 2022
@RuvendeGroote
fix: adds extra confirming of transactions
124c3c6 
Even when waiting for a transaction to hit a certain commitment \
level, we were noticing that calls that happen right after would \
sometimes return stale data.
If a transaction was included in slot #18 some of the calls would \
sometimes return data from that was present at slot #17
@RuvendeGroote RuvendeGroote mentioned this pull request Feb 7, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@BouweCeunen @RuvendeGroote @MaximPiessen