search

asyncapi-archived-repos / docgen

AsyncAPI documentation generator. DEPRECATED in favour of
https://github.com/asyncapi/generator
Apache License 2.0
21 stars 11 forks source link

Allow separator mixing #19

Closed cajus closed 6 years ago

cajus commented 6 years ago

In the current implementation, it is not possible to change the x-topic-separator to something else, without having this 'something else' beeing rewritten to the separator.

Example:

Setting x-topic-separator to '/' for MQTT style topics, it is not possible to get this topic in the documentation:

foo/devices/1.0/{serial}/join

It is always rewritten to:

foo/devices/1/0/{serial}/join

This PR changes this behaviour. Hopefully without shredding something else, because the npm tests are not existent ;-)

fmvilas commented 6 years ago

Thanks for taking the time @cajus. You're totally right. I was so used to the dots that I unconsciously attributed them a meaning. Thanks for fixing it.