Skip to content

Add transaction detail API cache#1979

Merged
lubej merged 1 commit intomasterfrom
ml/cache-tx-detail
Jul 11, 2024
Merged

Add transaction detail API cache#1979
lubej merged 1 commit intomasterfrom
ml/cache-tx-detail

Conversation

@lubej
Copy link
Copy Markdown
Contributor

@lubej lubej commented Jun 19, 2024

Adds cache for the following 2 endpoints:

  • getTransaction (/chain/transaction/{hash})
  • getRuntimeTransactionInfo (/runtime/transaction/info?id={paraTimeId}&hash={hash})

As those responses won't change, once indexed, they are cached through the app session(until user closes the wallet).

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 19, 2024

Deployed to Cloudflare Pages

Latest commit: 393024ea687ccfb5e62ddbd995cef7983a635606
Status:✅ Deploy successful!
Preview URL: https://db9dddee.oasis-wallet.pages.dev

@lubej lubej force-pushed the ml/cache-tx-detail branch from 09543ca to 48a691b Compare June 19, 2024 08:17
This was referenced Jun 19, 2024
@lubej lubej force-pushed the ml/cache-tx-detail branch 2 times, most recently from 3171098 to 8652661 Compare July 10, 2024 05:06
@lubej lubej changed the title Add getTransaction cache Add transaction detail API cache Jul 10, 2024
@lubej lubej marked this pull request as ready for review July 10, 2024 06:20
@lubej lubej requested review from buberdds and lukaw3d and removed request for lukaw3d July 10, 2024 06:32
Comment thread src/vendors/oasisscan.ts
Comment thread src/vendors/oasisscan.ts Outdated
Comment thread src/vendors/oasisscan.ts
@lubej lubej force-pushed the ml/cache-tx-detail branch 3 times, most recently from 34e91e5 to 658b603 Compare July 11, 2024 08:12
- as /chain/transaction/{hash} endpoint response won't change, cache it
@lubej lubej force-pushed the ml/cache-tx-detail branch from 658b603 to 393024e Compare July 11, 2024 08:26
@lubej lubej merged commit fafb9ca into master Jul 11, 2024
@lubej lubej deleted the ml/cache-tx-detail branch July 11, 2024 08:34
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

Successfully merging this pull request may close these issues.

2 participants