Skip to content
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

Fix number of messages in bp_messages.md #359

Merged
merged 2 commits into from Aug 17, 2019

Conversation

@codebrainz
Copy link
Contributor

commented Aug 17, 2019

I thought maybe the 25 was also including the Advanced messages, but there are only 4 of those and the surrounding text here clearly refers to the Basic messages. Or did I miss something?

@KSDaemon

This comment has been minimized.

Copy link
Collaborator

commented Aug 17, 2019

Maybe we can just omit the number of messages? I don't think it is a very important number, the word “all” is enough. What do you think?

@oberstet oberstet merged commit 5e9c2f8 into wamp-proto:master Aug 17, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.