-
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.
specifyweb/specify/views.py
Outdated
| 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.
alesan99
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.
I implemented jason's suggested changes to the setup PR so the API now works with this PR 👌
You're probably aware of most of these but I wanted to make note of them for reference:
- I think the spacing looks a bit weird between sections. I think collections need a bit less space, or everything should have the same amount of spacing.
-
An deletion blocker error appears (after a while) when opening a discipline or division. Specify 7 Crash Report - 2025-12-30T16_13_56.557Z.txt
-
Trees need to be created and linked to new disciplines somehow. Right now it works because its falling back to the last created trees.
-
v Icon, not sure which one should be used though.
| )} | ||
| </div> | ||
| {newResourceOpen ? ( | ||
| <Dialog |
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.

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