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

Add user id model #88

Merged
merged 2 commits into from
Aug 28, 2024
Merged

Add user id model #88

merged 2 commits into from
Aug 28, 2024

Conversation

SHAcollision
Copy link
Collaborator

This PR adds a basic UserID model as a wrapper of a String with capabilities to self-validate valid public keys. It replaces the String type on all of our user_id model fields.

Pre-submission Checklist

For tests to work you need a working neo4j instance with the example dataset in docker/db-migration

  • Testing: Implement and pass new tests for all new code, while maintaining existing test suite, cargo test.
  • Performance: Ensure new code has relevant performance benchmarks, cargo bench

@SHAcollision SHAcollision added the enhancement New feature or request label Aug 27, 2024
@SHAcollision SHAcollision requested a review from tipogi August 27, 2024 03:26
@SHAcollision SHAcollision self-assigned this Aug 27, 2024
@SHAcollision SHAcollision merged commit 4188212 into main Aug 28, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant