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

Use mapping from external AstarBase #39

Merged
merged 11 commits into from
Jul 19, 2022
Merged

Use mapping from external AstarBase #39

merged 11 commits into from
Jul 19, 2022

Conversation

Maar-io
Copy link
Member

@Maar-io Maar-io commented May 27, 2022

Since there are many already registered users in the initial AstarBase on Shiden, this PR will use this old contract as an external database.
If an entry is not found in the local contract, we will query the external (old) contract.
If the entry is found in the external contract, it will be registered locally. This is to same some gas and speed up tx next time the same entry is queried.

@github-actions
Copy link

github-actions bot commented May 27, 2022

Visit the preview URL for this PR (updated for commit 031ace1):

https://astarbase-a4922--pr39-external-astarbase-w0jwuqmp.web.app

(expires Tue, 26 Jul 2022 15:18:23 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

@Maar-io Maar-io merged commit 71b032b into main Jul 19, 2022
@Maar-io Maar-io deleted the external-astarbase branch July 19, 2022 15:29
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