-
Notifications
You must be signed in to change notification settings - Fork 41
Create a configuration tool #7312
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
base: issue-2931-1
Are you sure you want to change the base?
Conversation
e86d628 to
22ca396
Compare
11b3023 to
4a6da84
Compare
015e7e5 to
4396b60
Compare
dfd7ce5 to
4bace18
Compare
Triggered by 6f46015 on branch refs/heads/issue-6213-1
Discipline section: |
acwhite211
left a comment
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.
That should work, assuming we're ok with the models.CASCADE being added to those models.
| return http.HttpResponseBadRequest(e) | ||
| except http.Http404 as e: | ||
| return http.HttpResponseNotFound(e) | ||
| except IntegrityError as e: |
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.
to discuss
Triggered by 9061ca6 on branch refs/heads/issue-6213-1
acwhite211
left a comment
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.
Did more testing. The process of creating all the new records in the configuration tool works well!
We can work more on the UserGroupScopeId issue in the near future. Hopefully we can figure out a solution soon.
Fixes #6213
Checklist
self-explanatory (or properly documented)
Testing instructions
Notes: Taxon will be created for a discipline in the tree viewer not in the config tool