Skip to content
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

[POC] Mutations? ✅ Custom scalar support? ✅ #336

Merged
merged 1 commit into from
Sep 14, 2018
Merged

Conversation

eoji
Copy link
Contributor

@eoji eoji commented Sep 14, 2018

what

Added a POC for custom scalars.
Added adapter for Email custom scalar.
In the TestApolloActivity, logged in users can now update their email addresses.

how sway

Added Email, defined as a java.lang.String to customTypeMapping in build.gradle.
Added adapter for encoding and decoding Email in EmailAdapter. Should we do some validation here? It happens server side.
Added mutation for UpdateUserEmail that takes in email and current_password.
Updated TestApolloActivity to actually make this call.

image

Shouts to @orionmcc for support!

Added adapter for Email custom scalar.
In the `TestApolloActivity`, logged in users can now update their email addresses.
@eoji eoji requested a review from Rcureton September 14, 2018 21:08
update_email.text = "Update email"
update_email.setOnClickListener {
this.viewModel.inputs.updateEmailClicked(new_email.text(), current_password.text())
clearNameAndEmail()
Copy link
Contributor

Choose a reason for hiding this comment

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

I see you using that extension 😏

@eoji
Copy link
Contributor Author

eoji commented Sep 14, 2018

@eoji eoji merged commit f39092a into master Sep 14, 2018
@eoji eoji deleted the io/mutations branch September 14, 2018 21:40
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.

None yet

2 participants