Wrong remove mutations scheme

[xinhuang327]

Remove actions in the demo don't work as expected, removed items show up after GraphQL responses.

The reason I've found is in MutateAndGetPayload of RemoveCompletedTodos and RemoveTodo mutations, they should return GlobalIDs (like "VG9kbzoy") instead of integer IDs defined in database.

id, _ := inputMap["id"].(string)
resolvedId := relay.FromGlobalID(id)
RemoveTodo(resolvedId.ID)
return map[string]interface{}{
     "todoId": resolvedId.ID, // resolvedId.ID should be id.
}, nil

[sogko]

Hi @xinhuang327

Thanks for taking time to look into this, appreciate it very much.

I would very much be open to accept a PR on this if you want to contribute ☺️
(I've gotten my hands full at the moment, I'll work on this when I have the time to spare. In the mean time, any help would be very much appreciated! 😃)

Cheers!

---

Some notes:

• The code that @xinhuang327 mentions is here
• The corresponding code in relay-js todo example is here
• At first glance, the todo example in relay-js seems like it would have the same issue here. Might need to investigate that.