Skip to content

[OPAL-529] invoices summary page - Tax not included in final Amount#329

Open
MAhsenArif wants to merge 2 commits intomaestrano:2.0from
MAhsenArif:bug/529_include_tax_in_invoice
Open

[OPAL-529] invoices summary page - Tax not included in final Amount#329
MAhsenArif wants to merge 2 commits intomaestrano:2.0from
MAhsenArif:bug/529_include_tax_in_invoice

Conversation

@MAhsenArif
Copy link
Copy Markdown
Contributor

No description provided.

@adamaziz15 adamaziz15 self-requested a review August 22, 2018 20:02
@adamaziz15 adamaziz15 added the bug label Aug 22, 2018
@adamaziz15 adamaziz15 added this to the Opal SoW2 Wk 34 milestone Aug 22, 2018
Comment thread src/locales/en-AU.json
"mnoe_admin_panel.dashboard.invoice.expected_payment_date":"Expected payment date",
"mnoe_admin_panel.dashboard.invoice.amount":"Total amount",
"mnoe_admin_panel.dashboard.invoice.past_due": "Past Due",
"mnoe_admin_panel.dashboard.invoice.amount":"Total amount(excl. past due)",
Copy link
Copy Markdown

@alexmaestrano alexmaestrano Aug 22, 2018

Choose a reason for hiding this comment

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

Should there be a space between amount and (excl. past due)?

@ouranos
Copy link
Copy Markdown
Contributor

ouranos commented Aug 27, 2018

Do NOT merge. Discussion pending.

@MAhsenArif
Copy link
Copy Markdown
Contributor Author

Not merging. Just some edge cases that I found out about and were not pushed

@ouranos
Copy link
Copy Markdown
Contributor

ouranos commented Aug 27, 2018

Yes, that was more for @adamaziz15
The work on this should be put on hold until clarified

@ouranos ouranos modified the milestones: Opal SoW2 Wk 34, Opal SoW2 - Backlog Aug 31, 2018
@iseessel iseessel removed their request for review July 12, 2019 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants