Skip to content

Current order#79

Merged
ChristianRimondi merged 2 commits intomasterfrom
mdesantis/add-current-order
Mar 6, 2020
Merged

Current order#79
ChristianRimondi merged 2 commits intomasterfrom
mdesantis/add-current-order

Conversation

@mdesantis
Copy link
Copy Markdown

@mdesantis mdesantis commented Feb 14, 2020

It adds current_order to the context, which is responsible for fetching the last incomplete order from the current user or, if not found, the current order by the guest token. It returns nil if none of both are found.

It also adds the currentOrder query, which is responsible for fetching the current order form the context and returning it.

@mdesantis mdesantis force-pushed the mdesantis/add-current-order branch 5 times, most recently from e255f8a to 2600a38 Compare February 28, 2020 11:45
@mdesantis mdesantis self-assigned this Feb 28, 2020
@mdesantis mdesantis marked this pull request as ready for review February 28, 2020 12:02
@mdesantis mdesantis force-pushed the mdesantis/add-current-order branch from 2600a38 to e5af401 Compare February 28, 2020 12:06
@mdesantis mdesantis added the enhancement New feature or request label Feb 28, 2020
@mdesantis mdesantis force-pushed the mdesantis/add-current-order branch from e5af401 to cfeeb71 Compare February 28, 2020 13:37
@mdesantis mdesantis force-pushed the mdesantis/add-current-order branch 2 times, most recently from f6f912d to c022c7a Compare February 28, 2020 14:31
Maurizio De Santis added 2 commits February 28, 2020 15:34
It adds `current_order` to the context, which is responsible for
fetching the current order from the current user or, if not found, the
current order by the guest token. It returns `nil` if none of both are
found.
It returns the current order that is calculated in the context.
@mdesantis mdesantis force-pushed the mdesantis/add-current-order branch from c022c7a to 67259f4 Compare February 28, 2020 14:34
Copy link
Copy Markdown
Contributor

@AlessioRocco AlessioRocco left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Copy Markdown
Contributor

@ChristianRimondi ChristianRimondi left a comment

Choose a reason for hiding this comment

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

🎉

Copy link
Copy Markdown
Contributor

@rainerdema rainerdema left a comment

Choose a reason for hiding this comment

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

🔝

@ChristianRimondi ChristianRimondi merged commit d2a2916 into master Mar 6, 2020
@ChristianRimondi ChristianRimondi deleted the mdesantis/add-current-order branch March 6, 2020 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants