-
Notifications
You must be signed in to change notification settings - Fork 5
Backend draft #24
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
Backend draft #24
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.
Great work - read the comments for a few things to fix up :)
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.
🚢 Merge for now to move this along - but note you have some issues with LDAP also some of your DB commands aren't generating the correct items for the DB (e.g. newCat
)
Changes
Change reason
Closes
Checklist
Close #issue_number
git rebase -i origin/dev
(other relevant base branch). Read the contributing guide if you're unsure.