Skip to content
This repository has been archived by the owner on Feb 23, 2024. It is now read-only.

feat: cache receipts data #175

Merged
merged 22 commits into from
Oct 26, 2023
Merged

feat: cache receipts data #175

merged 22 commits into from
Oct 26, 2023

Conversation

matt-user
Copy link
Contributor

@matt-user matt-user commented Oct 24, 2023

closes FRO-670
closes #161

@linear
Copy link

linear bot commented Oct 24, 2023

FRO-670 tx list - cache receipts data for settled transactions

avoid first request by caching it if transaction is "settled"

Copy link
Contributor

@LuizAsFight LuizAsFight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should not cache everytime it passes through the gettingReceiptsInfo state

we should cache only when get to done state like it was before, but also include receipts data on cache.

then the state checkingDoneCache will be first step instead of second

@LuizAsFight
Copy link
Contributor

LuizAsFight commented Oct 26, 2023

this looks good, just need e2e tests to pass

@matt-user matt-user merged commit 6c46cf2 into main Oct 26, 2023
@matt-user matt-user deleted the matt-user/cache-receipts-data branch October 26, 2023 15:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

tx list - cache receipts data for settled transactions
2 participants