Skip to content

Set order email mutation#125

Merged
ChristianRimondi merged 3 commits intomasterfrom
set-order-email-mutation
Jun 26, 2020
Merged

Set order email mutation#125
ChristianRimondi merged 3 commits intomasterfrom
set-order-email-mutation

Conversation

@ChristianRimondi
Copy link
Copy Markdown
Contributor

Add a mutation to set the order's email. It accepts the email as input and sets it on the current order.

It raises a CanCan::AccessDenied exception in case the current ability can't update the current order.

@ChristianRimondi ChristianRimondi self-assigned this Jun 26, 2020
@ChristianRimondi ChristianRimondi force-pushed the set-order-email-mutation branch from 6d9b08c to 2e6ae1d Compare June 26, 2020 07:57
When creating the order as a guest user the email is not setted yet.
The mutation accepts the email as input and sets it on the current order.

It raises a CanCan::AccessDenied exception in case the current ability
can't update the current order.
Using command "bundle exec rake schema:idl"
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.

👍

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.

Looks good to me!

@ChristianRimondi ChristianRimondi merged commit 4af3ea9 into master Jun 26, 2020
@ChristianRimondi ChristianRimondi deleted the set-order-email-mutation branch June 26, 2020 10:42
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