-
Notifications
You must be signed in to change notification settings - Fork 0
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
Database initialisation function using RXdb #6
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pls see comments
Tick the correct box in the what type of change is this
section, add a gif and I can approve
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this script still apply? I thought that node can't run the db as it needs to run in the client?
@maxitect can you respond to Jack's comments? You made the most recent changes and I don't wana get it wrong
What type of PR is this? (check all applicable)
Description
Screenshots
Please replace this line with any relevant images for UI changes.
UI accessibility checklist
If your PR includes UI changes, please utilize this checklist:
Critical
andSerious
issues?Added/updated tests?
Please aim to keep the code coverage percentage at 80% and above.
What gif best describes this PR or how it makes you feel?