Skip to content

OBPIH-6781 prevent refreshing and deleting prepayment invoices that already have final invoices associated#4903

Merged
awalkowiak merged 3 commits intorelease/0.9.2-hotfix1from
feature/OBPIH-6781-prevent-refreshing-and-deleting-prepayment-invoices-that-already-have-final-invoices-associated
Oct 22, 2024
Merged

OBPIH-6781 prevent refreshing and deleting prepayment invoices that already have final invoices associated#4903
awalkowiak merged 3 commits intorelease/0.9.2-hotfix1from
feature/OBPIH-6781-prevent-refreshing-and-deleting-prepayment-invoices-that-already-have-final-invoices-associated

Conversation

@drodzewicz
Copy link
Collaborator

@drodzewicz drodzewicz commented Oct 21, 2024

✨ Description of Change

A concise summary of what is being changed. Please provide enough context for reviewers to be able to understand the change and why it is necessary. If the issue/ticket already provides enough information, you can put "See ticket" as the description.

Link to GitHub issue or Jira ticket: OBPIH-6781

Description:


📷 Screenshots & Recordings (optional)

If this PR contains a UI change, consider adding one or more screenshots here or link to a screen recording to help reviewers visualize the change. Otherwise, you can remove this section.

When trying to rollback prepayment invoice while regular invoice exists
image

When trying to delete prepayment invoice while regular invoice exists

image

@drodzewicz drodzewicz self-assigned this Oct 21, 2024
@github-actions github-actions bot added type: feature A new piece of functionality for the app domain: frontend Changes or discussions relating to the frontend UI domain: backend Changes or discussions relating to the backend server domain: l10n Changes or discussions relating to localization & Internationalization labels Oct 21, 2024
@drodzewicz drodzewicz changed the title Feature/obpih 6781 prevent refreshing and deleting prepayment invoices that already have final invoices associated OBPIH-6781 prevent refreshing and deleting prepayment invoices that already have final invoices associated Oct 21, 2024
} else {
invoiceInstance.datePosted = null
if (params.refreshInvoice && invoiceInstance.isPrepaymentInvoice) {
invoiceService.refreshInvoiceItems(invoiceInstance)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this method is only used here, so it could be removed as well.

@drodzewicz drodzewicz requested a review from awalkowiak October 22, 2024 09:08
@awalkowiak awalkowiak merged commit 24e47b0 into release/0.9.2-hotfix1 Oct 22, 2024
@awalkowiak awalkowiak deleted the feature/OBPIH-6781-prevent-refreshing-and-deleting-prepayment-invoices-that-already-have-final-invoices-associated branch October 22, 2024 09:28
jwalbers pushed a commit to jwalbers/openboxes that referenced this pull request Oct 29, 2024
jwalbers pushed a commit to jwalbers/openboxes that referenced this pull request Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: backend Changes or discussions relating to the backend server domain: frontend Changes or discussions relating to the frontend UI domain: l10n Changes or discussions relating to localization & Internationalization type: feature A new piece of functionality for the app

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants