-
Notifications
You must be signed in to change notification settings - Fork 18
Added new StatusLight component
#1804
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
|
Only the basic documentation for the website was done. |
|
An observation on this PR and for the next time. When we develop a new component we normally do the implementation first and then the documentation in a separate PR. This alleviates the burden of the review, allows the task to be broken down into smaller modules and also has the benefit that once the implementation has been merged, a new version of the library is generated and can therefore be used in the documentation PR for the examples. |
|
Also, developers are only in charge of the |
Checklist
(Check off all the items before submitting)
/libdirectory./websiteas needed.Description
Implementing the new
StatusLightcomponent, including testing and documentation.Closes #1803