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

Add (T, U, V) triple primary key #197

Closed
ethanfrey opened this issue Dec 21, 2020 · 0 comments · Fixed by #210
Closed

Add (T, U, V) triple primary key #197

ethanfrey opened this issue Dec 21, 2020 · 0 comments · Fixed by #210
Assignees
@maurolacy
Labels
storage plus
Projects
Contract development

Comments

@ethanfrey
Copy link
Member

ethanfrey commented Dec 21, 2020 •
edited

This makes very clear the distinction between prefix and suffix.

We should be able to range by fixing (T, U) and iterating over V

Let's test both the following: ((T, U), V) and (T, U, V). For both, we should be able to give (T, U) and iterate over V.

Example: (&[u8], U32Key, U64Key)
@ethanfrey ethanfrey added this to To do in Contract development via automation Dec 21, 2020
@ethanfrey ethanfrey mentioned this issue Dec 21, 2020
Closed
@maurolacy maurolacy self-assigned this Dec 24, 2020
@maurolacy maurolacy moved this from To do to In progress in Contract development Dec 24, 2020
@maurolacy maurolacy mentioned this issue Dec 28, 2020
Merged
@maurolacy maurolacy moved this from In progress to Review in progress in Contract development Dec 28, 2020
@maurolacy maurolacy mentioned this issue Dec 28, 2020
Merged
Contract development automation moved this from Review in progress to Done Jan 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maurolacy maurolacy

Labels
storage plus
Projects
No open projects
Contract development
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement PrimaryKey for generic (T, U, V) triplet CosmWasm/cw-plus
2 participants
@ethanfrey @maurolacy