Skip to content

OBPIH-6500 frontend modifications to the invoice confirmation page#4780

Closed
drodzewicz wants to merge 6 commits intodevelopfrom
OBPIH-6500-frontend-modifications-to-the-invoice-confirmation-page
Closed

OBPIH-6500 frontend modifications to the invoice confirmation page#4780
drodzewicz wants to merge 6 commits intodevelopfrom
OBPIH-6500-frontend-modifications-to-the-invoice-confirmation-page

Conversation

@drodzewicz
Copy link
Collaborator

@drodzewicz drodzewicz commented Aug 6, 2024

✨ Description of Change

Link to GitHub issue or Jira ticket: OBPIH-6500

Description:
I have split some of the components used in the Invoice create workflow and changed them to functional components.
Also separated the create components from list components for clarity.

I have also created two separate invoice item tables, one for regular invoices and the other for prepared which are rendered conditioanlly depending on what kind of an invoice we are dealing with

I have also removed pulling of prepayed items since we are gonna get served all of the items from a single /items endpoint.

Since the endpoint is not yet finished I was not able to go to far, and there is still quite some work left to be done.
The next steps to do here:

  1. since there are two tables, we need to extract inverse items from the endpoint and pass it to the table. (or we can serve them in a single list)
  2. I have not implemented logic for adding styling to the canceled item

@drodzewicz drodzewicz added the status: work in progress For issues or pull requests that are in progress but not yet completed label Aug 6, 2024
@github-actions github-actions bot added the domain: frontend Changes or discussions relating to the frontend UI label Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

domain: frontend Changes or discussions relating to the frontend UI status: work in progress For issues or pull requests that are in progress but not yet completed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants