Update struct for notification request#19
Update struct for notification request#19hrishavkmr wants to merge 4 commits intofossapps:masterfrom
Conversation
| IOSContentAvailable bool `json:"content_available"` | ||
| IOSNotification IOSNotification `json:"notification"` | ||
| To []string `json:"to"` | ||
| Data map[string]interface{} `json:"data"` |
There was a problem hiding this comment.
There's a good chance that some people might not actually use JSON, that's the reason I didn't want to have map, if you have a map, you need to parse into text and send that to pushy. What if they want to send an XML, may I know the reason for this?
also build failed.
There was a problem hiding this comment.
pushyMessage := pushy.SendNotificationRequest{
To: []string{<device_id>},
Data: `{"message":"hello world"}`,
}
I have tried using it like this, this doesn't work with pushy API. i get this error - Please supply the notification payload as a key-value object.
There was a problem hiding this comment.
you can fix the build, I'll test it and I'll merge if it works,
There was a problem hiding this comment.
or let me check this weekend if there's some new change in pushy's API, I was using this for a while and then I stopped doing android.
There was a problem hiding this comment.
The TravisCI build is failing for this reason - The command "go get -u golang.org/x/lint/golint" failed and exited with 2 during.
I think this is the same reason for which the last commit on the master was failing.
Any idea on how can we fix this?
2 participants
No description provided.