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

known_engine_accounts: Vec -> LookupSet #47

Closed
olga24912 opened this issue May 10, 2023 · 1 comment
Closed

known_engine_accounts: Vec -> LookupSet #47

olga24912 opened this issue May 10, 2023 · 1 comment
Assignees
Milestone

Comments

@olga24912
Copy link
Collaborator

We need to do the next operations with known_engine_accounts:

It looks like operations for the LookupSet, not Vec. On the other hand, usually this vector will only store one element. Therefore, it is possible using a vector here is more optimal. I was confused that we are using a regular vector, and not from near_sdk::collections

@mrLSD mrLSD self-assigned this May 15, 2023
@mrLSD mrLSD added this to the v0.5.1 milestone May 15, 2023
@mrLSD
Copy link
Collaborator

mrLSD commented May 15, 2023

Refactored at #50

@mrLSD mrLSD closed this as completed May 15, 2023
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

No branches or pull requests

2 participants