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

feat: make totalGave, totalGot, partialFill not undefined #1637

Conversation

Louis-Amas
Copy link
Contributor

@Louis-Amas Louis-Amas commented Nov 29, 2023

@Louis-Amas Louis-Amas closed this Nov 29, 2023
@Louis-Amas Louis-Amas reopened this Nov 29, 2023
@Louis-Amas Louis-Amas force-pushed the feat/louis/separate-clean-summary-from-order branch from 4e03e38 to 494899e Compare November 29, 2023 10:56
@Louis-Amas Louis-Amas force-pushed the feat/louis/add-totalgot-and-totalgave-to-order-summary branch from bc10005 to c928476 Compare November 29, 2023 10:58
@Louis-Amas Louis-Amas changed the base branch from feat/louis/separate-clean-summary-from-order to develop November 29, 2023 10:58
@Louis-Amas Louis-Amas closed this Nov 29, 2023
@Louis-Amas Louis-Amas reopened this Nov 29, 2023
…/louis/add-totalgot-and-totalgave-to-order-summary
@Louis-Amas Louis-Amas changed the base branch from develop to feat/louis/separate-clean-summary-from-order November 29, 2023 12:56
@Louis-Amas Louis-Amas closed this Nov 29, 2023
@Louis-Amas Louis-Amas reopened this Nov 29, 2023
Copy link
Contributor

@lnist lnist left a comment

Choose a reason for hiding this comment

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

This one seems to have compile errors?

@Louis-Amas Louis-Amas merged commit c50d4f9 into feat/louis/separate-clean-summary-from-order Nov 29, 2023
3 checks passed
@Louis-Amas Louis-Amas deleted the feat/louis/add-totalgot-and-totalgave-to-order-summary branch November 29, 2023 14:05
Louis-Amas added a commit that referenced this pull request Nov 29, 2023
* feat: separate-clean-summary-from-order

* fix: add removed tests

* feat: make totalGave, totalGot, partialFill not undefined (#1637)

* feat: make totalGave, totalGot, partialFill not undefined

* fix: compile errors

* feat: create a specific type for cleaning
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.

None yet

2 participants