octocat commented 7 years ago

Let's add some, shall we?

dmitshur-test commented 7 years ago

~~Is it just me, or is removeReaction mutation not working?~~

Edit: This issue has been fixed. removeReaction mutation works successfully for me now.

It seems to be documented completely in parallel to addReaction:

Both have the exact same input and return fields. The only difference is in the name and description.

When I do the addReaction mutation, it works as expected:

mutation AddReactionToIssue {
  addReaction(input:{subjectId:"MDU6SXNzdWUyMzEzOTE1NTE=",content:HOORAY}) {
    reaction {
      content
    }
    subject {
      id
    }
  }
}

{
  "data": {
    "addReaction": {
      "reaction": {
        "content": "HOORAY"
      },
      "subject": {
        "id": "MDU6SXNzdWUyMzEzOTE1NTE="
      }
    }
  }
}

But doing an analogous remove mutation fails with:

mutation RemoveReactionToIssue {
  removeReaction(input:{subjectId:"MDU6SXNzdWUyMzEzOTE1NTE=",content:HOORAY}) {
    reaction {
      content
    }
    subject {
      id
    }
  }
}

{
  "data": {
    "removeReaction": null
  },
  "errors": [
    {
      "message": "Could not resolve to a node with the global id of 'MDU6SXNzdWUyMzEzOTE1NTE='",
      "type": "NOT_FOUND",
      "path": [
        "removeReaction"
      ],
      "locations": [
        {
          "line": 32,
          "column": 3
        }
      ]
    }
  ]
}

Am I supposed to provide a different subjectId for remove mutation? I've tried the id and the databaseId of the reaction node itself, still same error.

rebolek commented 7 years ago

testing comment

rebolek commented 7 years ago

testing comment

rebolek commented 7 years ago

testing comment

rebolek commented 7 years ago

testing comment

rebolek commented 7 years ago

testing comment

rebolek commented 7 years ago

testing comment

rebolek commented 7 years ago

testing comment

rebolek commented 7 years ago

testing comment

rebolek commented 7 years ago

A shiny new comment! :tada:

brianamarie commented 7 years ago

HEART

AshoneA commented 7 years ago

hello

polyipseity commented 6 years ago

No comment.

maple3142 commented 6 years ago

test comment from graphiql

shujianbu commented 6 years ago

Test.

daeheonjeon commented 6 years ago

test comment!

daeheonjeon commented 6 years ago

Fantastic!

magicds commented 6 years ago

add a comment

magicds commented 6 years ago

add a comment

magicds commented 6 years ago

add another comment

zuTTers commented 6 years ago

test comment

Ausgezeichnet commented 6 years ago

@octocat Why should we react to the same words in different ways?

Shuumatsu commented 6 years ago

test

vkkis93 commented 6 years ago

it works

Ausgezeichnet commented 6 years ago

@octocat

vkkis93 commented 6 years ago

it works

PixeLInc commented 6 years ago

owo what's this? :o