Skip to content

'url-args' is required for safari website notifications #129

Closed
wants to merge 3 commits into from

2 participants

@argon
Owner
argon commented Oct 14, 2013

Thanks for pointing this out. To keep it inline with the rest of the module this change also requires the appropriate setters as well as the accompanying docs. If you wish to add this I will include it, otherwise I will incorporate it in my next batch of work.

Thanks.

@icodeforlove

Updated the setters but i haven't update the documentation, because the url-args are web payload specific.

@argon
Owner
argon commented Oct 16, 2013

That looks good for me so far. The option should still be documented - that the method exists, and preferably a link to the apple doc page discussing the url-args parameters.

@argon
Owner
argon commented Oct 19, 2013

Included in v1.4.2

@argon argon closed this Oct 19, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.