@alloy => Add an updateConversation mutation for buyer_outcome updates #583

Merged
merged 3 commits into from Mar 29, 2017

Conversation

Projects
None yet
2 participants
@mzikherman
Contributor

mzikherman commented Mar 29, 2017

screen shot 2017-03-29 at 4 52 25 pm

Adds a mutation in order to set buyer outcome for a conversation.

Still adding specs.

@mzikherman

This comment has been minimized.

Show comment
Hide comment
@mzikherman

mzikherman Mar 29, 2017

Contributor

Updated with spec.

Contributor

mzikherman commented Mar 29, 2017

Updated with spec.

+ },
+ resolve: (root, { buyer_outcome, id }, request, { rootValue: { accessToken } }) => {
+ if (!accessToken) return null;
+ return gravity.with(accessToken, { method: 'POST' })('me/token', {

This comment has been minimized.

@alloy

alloy Mar 29, 2017

Member

We should definitely also create loaders for these that are pre-authenticated before passing them into the rootValue object.

@alloy

alloy Mar 29, 2017

Member

We should definitely also create loaders for these that are pre-authenticated before passing them into the rootValue object.

@alloy

This comment has been minimized.

Show comment
Hide comment
@alloy

alloy Mar 29, 2017

Member

Huh, somehow I had imagined that these would need to work with the Node protocol and the Global Object Identification spec, but I guess the ID can be of any type or form ¯_(ツ)_/¯

👨‍🎨

Member

alloy commented Mar 29, 2017

Huh, somehow I had imagined that these would need to work with the Node protocol and the Global Object Identification spec, but I guess the ID can be of any type or form ¯_(ツ)_/¯

👨‍🎨

@alloy alloy merged commit 6779feb into artsy:master Mar 29, 2017

1 check passed

semaphoreci The build passed on Semaphore.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment