-
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-1449: Implement createOrUpdateBackingAddress mutation #2074
MBL-1449: Implement createOrUpdateBackingAddress mutation #2074
Conversation
@testable import KsApi | ||
import XCTest | ||
|
||
final class ConfirmBackingAddressMutationTests: XCTestCase { |
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.
These tests are a little silly since they're just exercising the MockService
🤷♀️. But at least they provide a little documentation of what the Service call can do.
@@ -413,6 +413,9 @@ public protocol ServiceType { | |||
|
|||
func exchangeTokenForOAuthToken(params: OAuthTokenExchangeParams) | |||
-> AnyPublisher<OAuthTokenExchangeResponse, ErrorEnvelope> | |||
|
|||
/// Confirms a backer's address for a given backing. Returns a success boolean. | |||
func confirmBackingAddress(backingId: String, addressId: String) -> AnyPublisher<Bool, 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.
n.B. I'm just taking the ids
and inputs and outputting the success
boolean; seems much cleaner to me than making extra input/output objects.
6fb0bc7
to
32147b5
Compare
32147b5
to
aec34af
Compare
📲 What
Add the
createOrUpdateBackingAddress
to the app.🤔 Why
This mutation is used to confirm a backer's address for a given backing; it will be used in the PPO dashboard.