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

Upgrade chain-signature contract collections::LookupMap to store::LookupMap #619

Open
ChaoticTempest opened this issue May 29, 2024 · 3 comments
Open

Upgrade chain-signature contract collections::LookupMap to store::LookupMap #619

ChaoticTempest opened this issue May 29, 2024 · 3 comments
Labels
Emerging Tech Emerging Tech flying formation at Pagoda Near BOS NEAR BOS team at Pagoda

Comments

@ChaoticTempest
Copy link
Member

Minor optimization we can do is to use the store::LookupMap since collections will more than likely be deprecated in the future. near-sdk currently requires the legacy feature which isn't a big deal for now, but can be an issue later if it was removed from the sdk
@ChaoticTempest ChaoticTempest added Near BOS NEAR BOS team at Pagoda Emerging Tech Emerging Tech flying formation at Pagoda labels May 29, 2024
@ChaoticTempest ChaoticTempest mentioned this issue May 29, 2024
Merged
@duonghb53
Copy link

Can I get this issue?

@volovyks
Copy link
Collaborator

volovyks commented Jun 4, 2024

Hey @duonghb53 ! The contract is under heavy refactoring ATM. Let's see if this issue remains after it's done.
We are happy to see your contributions.

@duonghb53
Copy link

@volovyks yes, sure. I will wait.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Emerging Tech Emerging Tech flying formation at Pagoda Near BOS NEAR BOS team at Pagoda
Projects
Emerging Technologies
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
3 participants
@volovyks @ChaoticTempest @duonghb53