search

marionettejs / backbone.radio

Messaging patterns for Backbone applications.
MIT License
490 stars 50 forks source link

Updated README.md #270

Closed Bartosz-D3V closed 7 years ago

Bartosz-D3V commented 7 years ago

Updated minor issue regarding API documentation. Context and arguments shall be preceded by a coma i.e. this.channel.trigger('someEvent', [arg1, arg2, arg3, etc.]);

Many thanks

paulfalgout commented 7 years ago

Ahh.. really sorry.. so now that I'm looking at the actual issue, I don't think it is a mistake.. the [ ] is indicating that it's an optional argument, and not that it's expecting an array. hmm..

paulfalgout commented 7 years ago

Essentially this: http://stackoverflow.com/questions/17132014/documentation-for-optional-parameters-in-javascript

Again, I initially agreed.. sorry I didn't look closer earlier.

Still, 👏 thanks for contributing.