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

Invoices #570

Merged
merged 28 commits into from
Jul 8, 2020
Merged

Invoices #570

merged 28 commits into from
Jul 8, 2020

Conversation

orzechdev
Copy link
Contributor

@orzechdev orzechdev commented Jun 19, 2020

I want to merge this change because it adds invoices implementation into the dashboard.

PR intended to be tested with API branch:

saleor/saleor#5732

Screenshots

Pull Request Checklist

  1. All visible strings are translated with proper context.
  2. All data-formatting is locale-aware (dates, numbers, and so on).
  3. Translated strings are extracted.
  4. Number of API calls is optimized.
  5. The changes are tested.
  6. Type definitions are up to date.
  7. Changes are mentioned in the changelog.

@orzechdev orzechdev closed this Jun 23, 2020
@orzechdev orzechdev changed the title Create invoice cart in order deatials view Invoices Jun 23, 2020
@orzechdev orzechdev reopened this Jun 30, 2020
@orzechdev orzechdev force-pushed the feature/invoices branch 2 times, most recently from ba9b40e to 8e10ab2 Compare July 6, 2020 12:13
@dominik-zeglen dominik-zeglen force-pushed the feature/invoices branch 2 times, most recently from d5ddb05 to bffaf92 Compare July 7, 2020 15:22
@codecov
Copy link

codecov bot commented Jul 7, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@3d91ca1). Click here to learn what that means.
The diff coverage is 57.14%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #570   +/-   ##
=========================================
  Coverage          ?   71.51%           
=========================================
  Files             ?      725           
  Lines             ?     8327           
  Branches          ?     1378           
=========================================
  Hits              ?     5955           
  Misses            ?     2111           
  Partials          ?      261           
Impacted Files Coverage Δ
...ainers/BackgroundTasks/BackgroundTasksProvider.tsx 72.22% <0.00%> (ø)
src/intl.ts 100.00% <ø> (ø)
...s/components/OrderDetailsPage/OrderDetailsPage.tsx 92.00% <ø> (ø)
src/orders/urls.ts 45.00% <ø> (ø)
src/storybook/stories/orders/OrderDetailsPage.tsx 72.00% <0.00%> (ø)
...ebhooks/components/WebhookEvents/WebhookEvents.tsx 81.81% <ø> (ø)
src/containers/BackgroundTasks/tasks.ts 52.17% <10.00%> (ø)
src/storybook/stories/orders/OrderInvoiceList.tsx 62.50% <62.50%> (ø)
...s/components/OrderInvoiceList/OrderInvoiceList.tsx 85.00% <85.00%> (ø)
src/fragments/errors.ts 100.00% <100.00%> (ø)
... and 729 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3d91ca1...5b69669. Read the comment docs.

@dominik-zeglen dominik-zeglen merged commit 820a602 into master Jul 8, 2020
@dominik-zeglen dominik-zeglen deleted the feature/invoices branch July 8, 2020 08:56
@dominik-zeglen dominik-zeglen mentioned this pull request Jul 8, 2020
7 tasks
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.

3 participants