-
Notifications
You must be signed in to change notification settings - Fork 178
Move 6LoWPAN mesh configuration to its own page under Reference/Config #502
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
Conversation
I like the intro section on the config page. Please use the command-line to generate the configuration parameters, so they match the format of the other config pages. |
I did not spot any instructions to for any script. Also, I don't really like the way other configuration pages are structured. The script generated lists only contain name of the configuration key, no description of its use, not range of acceptable values. Can I still keep the tables and append the script generated lists after them? I originally expressed my concerns here: #366 Therefore, I wan't to have intro sections and tables listing the keys, acceptable values/ranges and use cases for keys. |
@SeppoTakalo See https://github.com/ARMmbed/Handbook/blob/new_engine/docs/reference/configuration/configuration.md#examining-available-configuration-parameters
If there is no description of it's use, then the |
@theotherjimmy Thanks. And yes, you are correct. Help fields are empty. I'll move the stuff to help section and create PR to the Mbed OS |
Refactored the page to look like other configuration pages. |
Publishing the changed also in Mbed OS ARMmbed/mbed-os#6745 |
😃 Thanks @SeppoTakalo |
Copy edit for active voice and formatting.
Copy edit changes, mostly for active voice and inclusion of articles.
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.
Awesome work on this 👍
@SeppoTakalo Can I make these changes to 5.8, or are they only for 5.9? |
These are valid for 5.8 as well. |
Update MeshInterface from #502 after reviewing on the test site.
@AnotherButler Please review.