-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
MBL-1177: Add CompleteOnSessionCheckoutMutation to GraphAPI #1981
MBL-1177: Add CompleteOnSessionCheckoutMutation to GraphAPI #1981
Conversation
d342558
to
ece7313
Compare
|
||
extension GraphAPI.CompleteOnSessionCheckoutMutation.Data: Decodable { | ||
public init(from _: Decoder) throws { | ||
fatalError("The test code should not actually be decoding this object.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is required for us to use performWithResult
. I'm fairly sure the Decodable
requirement is for Apollo, not for the MockGraphQLClient
, which doesn't actually do any decoding.
@@ -77,6 +77,9 @@ public protocol ServiceType { | |||
func createBacking(input: CreateBackingInput) -> | |||
SignalProducer<CreateBackingEnvelope, ErrorEnvelope> | |||
|
|||
func completeOnSessionCheckout(input: GraphAPI.CompleteOnSessionCheckoutInput) -> | |||
SignalProducer<GraphAPI.CompleteOnSessionCheckoutMutation.Data, ErrorEnvelope> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like to move away from wrapper objects unless we really need them - implementing this as its raw GraphQL types for now; if we need to write a wrapper later, I'm OK doing that just-in-time.
ece7313
to
b09f77f
Compare
b09f77f
to
567dec9
Compare
📲 What
Add CompleteOnSessionCheckoutMutation to GraphAPI.
🤔 Why
This is a new mutation for post-campaign pledging. I modeled it after the mutation it will replace,
createBacking
, having it return aCheckoutFragment
.👀 See
Trello, screenshots, external resources?