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

TP2000-1619-Sub-quota-create-volume-bug #1361

Merged
merged 10 commits into from
Dec 17, 2024

Conversation

mattjamc
Copy link
Collaborator

@mattjamc mattjamc commented Dec 16, 2024

TP2000-1619-Sub-quota-create-volume-bug

Why

Ivona encountered an error where she could not create new quota associations due to the sub-quota definition having a different volume to other sub-quotas of the same main-quota. Investigation discovered a few bugs with how the QA5 business rule is executed.

What

The following issues and fixes were identified:

  • The business rule was only comparing volumes of existing definitions so Ivona's first differing-volume creation didn't matter. It was the second one that triggered that issue. Solved by comparing the newly inputted volume with the existing one
  • It was using volume rather than initial volume which is why there was a difference in the first place. Fixed by changing to initial volume
  • original_definition.sub_quotas was including deleted associations which is why Ivona deleting it didn't fix the issue. Solved by adding .current() to the queryset
  • Removal of QA5 and QA6 checks for the SubQuotaAssociationDefinitionUpdateForm as it does not make sense to do checks in-form which check against other sub-quotas. It would block the user from letting 1 sub-quota differ even if their intention is to update the other ones afterwards.

@mattjamc mattjamc changed the title Test solution TP2000-1619-Sub-quota-create-volume-bug Dec 16, 2024
@mattjamc mattjamc marked this pull request as ready for review December 17, 2024 10:35
@mattjamc mattjamc requested a review from a team as a code owner December 17, 2024 10:35
@mattjamc mattjamc merged commit 26e153d into master Dec 17, 2024
8 checks passed
@mattjamc mattjamc deleted the TP2000-1619-Sub-quota-create-volume-bug branch December 17, 2024 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants