-
Notifications
You must be signed in to change notification settings - Fork 36
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
Open interaction dialog when adding prep in ExchangeOut #4805
Conversation
Triggered by 45c17ae on branch refs/heads/issue-922
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.
Currently, ExchangeOut
is not the only Interaction missing a dialog: ExchangeIn
is missing one as well!
I have some suggestions to make our current approach more generic without relying on hardcoding the tables which are supported.
specifyweb/frontend/js_src/lib/components/FormSliders/IntegratedRecordSelector.tsx
Outdated
Show resolved
Hide resolved
specifyweb/frontend/js_src/lib/components/FormSliders/IntegratedRecordSelector.tsx
Outdated
Show resolved
Hide resolved
specifyweb/frontend/js_src/lib/components/Interactions/InteractionDialog.tsx
Outdated
Show resolved
Hide resolved
specifyweb/frontend/js_src/lib/components/Interactions/PrepDialog.tsx
Outdated
Show resolved
Hide resolved
specifyweb/frontend/js_src/lib/components/Interactions/PrepDialog.tsx
Outdated
Show resolved
Hide resolved
specifyweb/frontend/js_src/lib/components/Interactions/PrepDialog.tsx
Outdated
Show resolved
Hide resolved
specifyweb/frontend/js_src/lib/components/Interactions/InteractionsDialog.tsx
Outdated
Show resolved
Hide resolved
Adds support for associating Preparations to ExchangeIn records via an Interaction Dialog
Triggered by e76e4ff on branch refs/heads/issue-922
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.
Testing instructions
- Go to interactions
- Open Exchange Out
- Add a prep
- verify the interaction dialog opens
- verify that you can add a prep by selecting "By choosing a record set"
- verify that you can add a prep by selecting "By entering catalog number"
- verify that you can add an unassociated item
- test adding a preparation for ExchangeIn
There is no option for adding an unassociated item when creating an exchange in/out, only in the add items dialog.
Also although you can add preparations through record set or catalog numbers in the create exchange in/out, when you try to add any items after that it just adds a blank preparation
LK8UmhhIet.mp4
Basically, the initial dialog works aside from missing the add unassociated preps button but the add items dialog will only add blank preparations.
Also side note, the exchange in default form does not include the preparations subview, is there a reason for that?
Triggered by 5641349 on branch refs/heads/issue-922
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.
Testing instructions
- Go to interactions
- Open Exchange Out
- Add a prep
- verify the interaction dialog opens
- verify that you can add a prep by selecting "By choosing a record set"
- verify that you can add a prep by selecting "By entering catalog number"
- verify that you can add an unassociated item
- test adding a preparation for ExchangeIn
Looks good, the without preparation button is there and it adds items correctly now!
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.
- verify the interaction dialog opens
- verify that you can add a prep by selecting "By choosing a record set"
- verify that you can add a prep by selecting "By entering catalog number"
- verify that you can add an unassociated item
- test adding a preparation for ExchangeIn
Checks out 👍
I also found it weird that there is no preparation subview in the default Exchange In form, but perhaps that can be addressed elsewhere
Fixes #922
Checklist
and self-explanatory (or properly documented)
Testing instructions