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

refresh token lists during session #4573

Draft
wants to merge 24 commits into
base: develop
Choose a base branch
from
Draft

refresh token lists during session #4573

wants to merge 24 commits into from
+577 −233

Conversation

emilydrakesmith
Copy link
Contributor

Describe your changes

Describe the purpose + content of these changes

Link the related issue

Closes #4572

Checklist before requesting a review

  • Is this PR ready for merge? (Please convert to a draft PR otherwise)
  • I have performed a self-review of my code.
  • Did I request feedback from a team member prior to the merge?
  • Does my code following the style guide at docs/CODING-STYLE.md?

Instructions for Reviewers

Functionalities or workflows that should specifically be tested.

Environmental conditions that may result in expected but differential behavior.

If relevant, list additional work to complete pre-merge (delete logging, code abstraction, etc)

@emilydrakesmith emilydrakesmith self-assigned this Dec 31, 2024
@netlify Netlify
Copy link

netlify bot commented Dec 31, 2024 •
edited
Loading

✅ Deploy Preview for dev-ambi ready!

Name Link
🔨 Latest commit 54d2be2
🔍 Latest deploy log https://app.netlify.com/sites/dev-ambi/deploys/677c5882a175cf0008dc7ed8
😎 Deploy Preview https://deploy-preview-4573--dev-ambi.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify Netlify
Copy link

netlify bot commented Dec 31, 2024 •
edited
Loading

✅ Deploy Preview for futa-finance ready!

Name Link
🔨 Latest commit 54d2be2
🔍 Latest deploy log https://app.netlify.com/sites/futa-finance/deploys/677c5882f6d489000899aa8b
😎 Deploy Preview https://deploy-preview-4573--futa-finance.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify Netlify
Copy link

netlify bot commented Dec 31, 2024 •
edited
Loading

✅ Deploy Preview for base-ambient-finance ready!

Name Link
🔨 Latest commit 54d2be2
🔍 Latest deploy log https://app.netlify.com/sites/base-ambient-finance/deploys/677c58820b0e0a00085cc26d
😎 Deploy Preview https://deploy-preview-4573--base-ambient-finance.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@emilydrakesmith emilydrakesmith changed the title add refresh interval data to tokenListURIs.ts file refresh token lists during session Dec 31, 2024
@emilydrakesmith emilydrakesmith added the enhancement New feature or request label Dec 31, 2024
emilydrakesmith and others added 22 commits December 31, 2024 11:26
@emilydrakesmith
update annotations in useTokens.ts file
1ac75cb
@emilydrakesmith
rescope fn fetchAndFormatList()
61b13ce
@emilydrakesmith
retype uri param in fn fetchAndFormatList()
01d749f
@emilydrakesmith
add new logic to patch an updated list into local state
4aff7b0
@benwolski
add local server for futa token list
622837a
@emilydrakesmith
expand list patching logic to patch multiple lists in parallel
e5dbd29
@emilydrakesmith
Merge branch 'refresh-token-lists' of https://github.com/CrocSwap/amb…
ffbdbec 
…ient-ts-app into refresh-token-lists
@benwolski
simplify fetching of futa list on interval
3d7de6d
@benwolski
Merge branch 'develop' into refresh-token-lists
8c64138
@benwolski
Merge branch 'develop' into refresh-token-lists
4d836cc
@emilydrakesmith
clean up FUTA token list refresh logic
aed9dd0
@emilydrakesmith
Merge branch 'refresh-token-lists' of https://github.com/CrocSwap/amb…
8e95f76 
…ient-ts-app into refresh-token-lists
@benwolski
add isFuta to TokenIF
0047587
@emilydrakesmith
Merge branch 'develop' of https://github.com/CrocSwap/ambient-ts-app …
7f25d4c 
…into refresh-token-lists
@emilydrakesmith
add logic to return only FUTA tokens from universw
c1fc67e
@emilydrakesmith
refactor default token handling logic
f3aaf97
@benwolski
Merge branch 'develop' into refresh-token-lists
bc64b56
@emilydrakesmith
float native token to top of FUTA token select modal
f2dd9a3
@emilydrakesmith
Merge branch 'refresh-token-lists' of https://github.com/CrocSwap/amb…
43239fc 
…ient-ts-app into refresh-token-lists
@emilydrakesmith
Merge branch 'develop' of https://github.com/CrocSwap/ambient-ts-app …
d45cfc0 
…into refresh-token-lists
@benwolski
add defaultTokensAmbient export
daef210
@emilydrakesmith
add special handling to get local token lists
d3dfd94
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@benwolski benwolski Awaiting requested review from benwolski

At least 1 approving review is required to merge this pull request.

Assignees

@emilydrakesmith emilydrakesmith

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Enhancement]: refresh token lists in-session
2 participants
@emilydrakesmith @benwolski