Skip to content
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

Prevent changing storage location on itemizables if there has been an intervening audit of the items involved #4304

Open
2 tasks
cielf opened this issue Apr 25, 2024 · 4 comments

Comments

@cielf
Copy link
Collaborator

cielf commented Apr 25, 2024

Summary

Prevent changing storage location on donation/distribution/purchase if there has been an intervening audit on the items involved.

Why?

It's an extremely edge case, and we're not sure what the effects on inventory should be in this case. It gets really messy if new items are added at the same time, for example!

Details

For each of purchase, donation, and distribution:
On edit, on save, if the storage location has been changed, and there has been an intervening audit on any of the items involved (either original or new) for either of the storage locations, give the error message as follows:

For distribution: "Cannot change the storage location because there has been an intervening audit of some items. If you need to change the storage location, please reclaim this distribution and create a new distribution from the new storage location"

For donation: "Cannot change the storage location because there has been an intervening audit of some items. If you need to change the storage location, please delete this donation and create a new donation with the new storage location"

For purchases: "Cannot change the storage location because there has been an intervening audit of some items. If you need to change the storage location, please delete this purchase and create a new purchase with the new storage location"

Criteria for completion

  • functionality as described above
  • tests to support functionality as described above
@jp524
Copy link
Contributor

jp524 commented May 28, 2024

I'd like to help with this issue.

@cielf
Copy link
Collaborator Author

cielf commented May 28, 2024

Please do!

@github-actions github-actions bot removed the Help Wanted Groomed + open to all! label May 28, 2024
@jp524
Copy link
Contributor

jp524 commented May 29, 2024

@cielf I started looking at this and I saw that a warning is displayed on the edit pages for donation/distribution/purchase that have an intervening audit (please refer to #4026 and #4086).

I just wanted to confirm with you if the feature for this issue is to be added on top of the existing warning, or if the existing warning needs to be modified at all.

@cielf
Copy link
Collaborator Author

cielf commented May 30, 2024

Yes please, we still need it. Because someone could ignore the warning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants