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.

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

Improve performance when displaying token balance list #762

Merged
merged 4 commits into from
Aug 1, 2023

Conversation

csillag
Copy link
Contributor

@csillag csillag commented Jul 20, 2023

This depends on oasisprotocol/nexus#492.
This also depends on #761. (Since the Nexus PRs also depend on each other.)

Please note that while this version is backward compatible with the currently deployed staging or production versions of nexus, the performance improvements will only be realized if the new version of nexus is deployed.

@github-actions
Copy link

github-actions bot commented Jul 20, 2023

Deployed to Cloudflare Pages

Latest commit: 36d8bd05842f66d537bad406f57ef77d02690e4a
Status:✅ Deploy successful!
Preview URL: https://7aa3dcf3.oasis-explorer.pages.dev

@csillag csillag force-pushed the csillag/optimize-token-balance-list branch 3 times, most recently from 2bc2980 to cee9430 Compare July 20, 2023 16:33
@csillag csillag marked this pull request as ready for review July 20, 2023 16:36
@csillag csillag requested review from lukaw3d and buberdds July 20, 2023 16:36
@csillag csillag force-pushed the csillag/optimize-token-balance-list branch from cee9430 to a70fbf9 Compare July 20, 2023 23:01
@csillag
Copy link
Contributor Author

csillag commented Jul 21, 2023

Tested with latest staging Nexus deployment, seems to work fine. (For testing, I had to change the API endpoints to staging, and enable Emerald.)

@csillag csillag force-pushed the csillag/optimize-token-balance-list branch 4 times, most recently from d1b9bdb to 7e52f9d Compare August 1, 2023 02:13
@csillag
Copy link
Contributor Author

csillag commented Aug 1, 2023

Now that the relevant Nexus changes have been merged, this PR has also been updated accordingly.

@csillag csillag force-pushed the csillag/optimize-token-balance-list branch from 7e52f9d to 36d8bd0 Compare August 1, 2023 15:56
@csillag csillag merged commit 82f9ffa into master Aug 1, 2023
@csillag csillag deleted the csillag/optimize-token-balance-list branch August 1, 2023 15:59
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