Skip to content

OBPIH-6532 [frontend] Ability to edit invoice item#4798

Merged
awalkowiak merged 8 commits intofeature/OBPIH-6398-partial-invoices-for-prepaid-pofrom
OBPIH-6532
Aug 26, 2024
Merged

OBPIH-6532 [frontend] Ability to edit invoice item#4798
awalkowiak merged 8 commits intofeature/OBPIH-6398-partial-invoices-for-prepaid-pofrom
OBPIH-6532

Conversation

@alannadolny
Copy link
Collaborator

✨ 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: https://pihemr.atlassian.net/browse/OBPIH-6532

Description:

This PR is still in progress but ready for initial review. Before merging this one, merge the PR with the remove action.

@alannadolny alannadolny added status: work in progress For issues or pull requests that are in progress but not yet completed status: ready for review Flags that a pull request is ready to be reviewed labels Aug 22, 2024
@alannadolny alannadolny self-assigned this Aug 22, 2024
@github-actions github-actions bot added domain: frontend Changes or discussions relating to the frontend UI domain: backend Changes or discussions relating to the backend server labels Aug 22, 2024
@alannadolny alannadolny added the warn: do not merge Marks a pull request that is not yet ready to be merged label Aug 22, 2024
return invoice?.isPrepaymentInvoice
}

Integer getQuantityAvailableToInvoice() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could it be just:

return shipmentItem ? (shipmentItem.quantityToInvoiceInStandardUom / quantityPerUom) + quantity : null // or 0

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 there might be some comments needed in this hook, especially for revertQuantityToEdited

@alannadolny alannadolny removed status: work in progress For issues or pull requests that are in progress but not yet completed warn: do not merge Marks a pull request that is not yet ready to be merged labels Aug 23, 2024
@github-actions github-actions bot added the domain: l10n Changes or discussions relating to localization & Internationalization label Aug 26, 2024
@alannadolny alannadolny requested a review from ewaterman August 26, 2024 15:09
@awalkowiak awalkowiak merged commit c9e99ff into feature/OBPIH-6398-partial-invoices-for-prepaid-po Aug 26, 2024
@awalkowiak awalkowiak deleted the OBPIH-6532 branch August 26, 2024 16:52
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 status: ready for review Flags that a pull request is ready to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants