-
Notifications
You must be signed in to change notification settings - Fork 52
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
feat: added ability to cache genesis trans for 24h #1613
Conversation
Signed-off-by: wadeking98 <[email protected]>
Signed-off-by: wadeking98 <[email protected]>
Signed-off-by: wadeking98 <[email protected]>
Signed-off-by: wadeking98 <[email protected]>
Signed-off-by: wadeking98 <[email protected]>
Signed-off-by: wadeking98 <[email protected]>
Signed-off-by: wadeking98 <[email protected]>
Signed-off-by: wadeking98 <[email protected]>
Signed-off-by: wadeking98 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something seems odd about the one patch, everything else LGTM
Signed-off-by: wadeking98 <[email protected]>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
implemented caching for ledger genesis transactions.
The wallet will refresh the cache every 24 hours on load.
if the cache needs to be refreshes the wallet will refresh the pool and fetch the new genesis transactions. doing this also automatically updates the current agent transaction pool. The transactions are saved in local storage so we don't have to fetch them the next time we startup.
Note: this change also seems to improve performance with sorvin:staging network but more testing is needed
The following gifs were created with a local production build of the wallet:
Accept LSBC credential:
a2a proof request: