-
-
Notifications
You must be signed in to change notification settings - Fork 228
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tweak readme #78
tweak readme #78
Conversation
@@ -257,7 +257,6 @@ instance.authorizeSubscribe = function (client, sub, cb) { | |||
|
|||
It will be called when a client is set to recieve a message. Override to supply custom | |||
authorization logic. | |||
In order to completely pre | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A leftover sentence? Just removed it since I didn't know how to finish it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call
@@ -159,7 +159,7 @@ A packet must contain the following properties: | |||
} | |||
``` | |||
|
|||
Only the `topic` and properties are mandatory | |||
Only the `topic` and `payload` properties are mandatory. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this one was copy/pasted from mqtt-packet
repo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess payload
is missing?
@@ -147,7 +147,7 @@ It supports backpressure. | |||
|
|||
Publish the given packet to subscribed clients and functions. It supports backpressure. | |||
|
|||
A packet must contain the following properties: | |||
A packet contains the following properties: | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Contradicting sentences. This one states that all properties are mandatory but the second says only topic (and as I suspect, the payload property) is mandatory.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right
bump |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect, thanks!!
No description provided.