Skip to content

Add Adjustment types and the adjustments connection to the Order type#121

Merged
rainerdema merged 5 commits intomasterfrom
rainerd/add-adjustment-type
Jul 3, 2020
Merged

Add Adjustment types and the adjustments connection to the Order type#121
rainerdema merged 5 commits intomasterfrom
rainerd/add-adjustment-type

Conversation

@rainerdema
Copy link
Copy Markdown
Contributor

@rainerdema rainerdema commented Jun 12, 2020

This PR:

  • Creates the PromotionAdjustment and the TaxAdjustment types
  • Creates the Adjustment interface and implements it in the new types.
  • Adds the adjustments connection to the Order type

@rainerdema rainerdema self-assigned this Jun 12, 2020
@rainerdema rainerdema force-pushed the rainerd/add-adjustment-type branch 2 times, most recently from 8298f02 to 800057c Compare June 19, 2020 07:35
@rainerdema rainerdema force-pushed the rainerd/add-adjustment-type branch from 800057c to 7e71db5 Compare June 26, 2020 14:30
@rainerdema rainerdema changed the title Add Adjustment type Add Adjustment types and the adjustments connection to the Order type Jul 3, 2020
@rainerdema rainerdema force-pushed the rainerd/add-adjustment-type branch 5 times, most recently from 8f98c34 to a232c39 Compare July 3, 2020 05:49
@rainerdema rainerdema marked this pull request as ready for review July 3, 2020 06:09
It adds the `PromotionAdjustment` type, based on `Spree::PromotionAction`.
It adds the `TaxAdjustment` type, based on `Spree::TaxRate`.
It adds the `Adjustment` interface and implements it in the new
`TaxAdjustment` and `PromotionAdjustment` types.
These are declared as `orphan_types` because they are never used
directly but only connected through the `Adjustment` interface.

For more information about this:
https://graphql-ruby.org/type_definitions/interfaces.html#orphan-types
Using command "bundle exec rake schema:idl"
@rainerdema rainerdema force-pushed the rainerd/add-adjustment-type branch from a232c39 to a0cc481 Compare July 3, 2020 07:20
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

@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.

🚀

@rainerdema rainerdema merged commit 6b17e17 into master Jul 3, 2020
@rainerdema rainerdema deleted the rainerd/add-adjustment-type branch July 3, 2020 08:15
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