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

Refactor order input for comparable hashes #49

Open
j-ti opened this issue Jul 26, 2023 · 0 comments
Open

Refactor order input for comparable hashes #49

j-ti opened this issue Jul 26, 2023 · 0 comments

Comments

@j-ti
Copy link

j-ti commented Jul 26, 2023

Change cluster_index field to cluster_id
For the Order hash to be equivalent to the one accessible by the user both type of Order strings have to be equivalent:

  • the integer field cluster_index should become a String field cluster_id.
    • also needs to be changed in simplyr-lib dependency
  • to be clarified: should we integrate a field hash_version?
  • to be clarified: we might add a salt of type Char with length=32 to create non-guessable orders
@j-ti j-ti changed the title Change order_index field to order_id Refactor order input for comparable hashes Jul 31, 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

1 participant