-
Notifications
You must be signed in to change notification settings - Fork 11
Add TAPPER section #141
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
base: main
Are you sure you want to change the base?
Add TAPPER section #141
Conversation
✅ Deploy Preview for hardwario-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
d009bc8
to
60192d6
Compare
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.
Pull Request Overview
This PR adds a new documentation section for the TAPPER product, outlining its usage, installation, API specification, and introduction, along with updating the sidebar and Docusaurus configuration.
- Introduces a usage guide with command descriptions and configuration options
- Adds an introduction page outlining core features and product capabilities
- Provides installation instructions and MQTT API documentation, and updates sidebar and config files accordingly
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
tapper/usage.md | Adds client CLI usage instructions and configuration reference. |
tapper/introduction.md | Introduces TAPPER with features; contains minor spelling issues. |
tapper/installation.md | Provides detailed installation steps for the TAPPER client. |
tapper/api-spec.md | Documents the MQTT API specification for TAPPER. |
sidebars-tapper.js | Configures the sidebar for the TAPPER documentation section. |
docusaurus.config.js | Updates Docusaurus configuration to include the TAPPER docs. |
The checks failed on initialization - |
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.
Pull Request Overview
This PR adds a new documentation section for the TAPPER Product, detailing client usage, installation, API specifications, and an introduction.
- New markdown files for usage, introduction, installation, and API specification
- Integration of the TAPPER section into the Docusaurus sidebars and configuration
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
tapper/usage.md | New usage documentation for TAPPER CLI commands |
tapper/introduction.md | Introduction and overview of TAPPER features |
tapper/installation.md | Installation guide including prerequisites and steps |
tapper/api-spec.md | MQTT API specification details for TAPPER |
sidebars-tapper.js | Sidebar configuration for the TAPPER documentation |
docusaurus.config.js | Updated configuration to include the new tapper docs |
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.
Pull Request Overview
Adds the initial documentation and site configuration for the new TAPPER product section.
- Introduces four new docs: Introduction, Installation, Usage, and API Specification.
- Registers the TAPPER docs in the sidebar and adds a top‐level “TAPPER” nav entry.
- Updates
docusaurus.config.js
to include the new docs plugin.
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
tapper/usage.md | Add CLI usage instructions and configuration snippet |
tapper/introduction.md | Add project overview and feature list |
tapper/installation.md | Add step‐by‐step installation guide for Raspberry Pi |
tapper/api-spec.md | Add detailed MQTT API specification with topics and payloads |
sidebars-tapper.js | Define sidebar entries for the TAPPER docs |
docusaurus.config.js | Register the TAPPER docs plugin and add a “TAPPER” navbar link |
Comments suppressed due to low confidence (2)
tapper/introduction.md:16
- [nitpick] The indent on this bullet is inconsistent with the others. Remove the extra spaces so it aligns with the list.
- TLS connection is currently work-in-progress
docusaurus.config.js:106
- The new docs plugin block needs its
editUrl
indented inside the same object (like the apps plugin) so that the “Edit this page” link works for Tapper docs.
sidebarPath: require.resolve('./sidebars-tapper.js'),
Signed-off-by: Martin <81148675+Martian-0007@users.noreply.github.com>
- Add Installation - Add Usage - Rewrite Introduction - Moved MQTT API to a separate file, this is yet to be finished Signed-off-by: Martin <81148675+Martian-0007@users.noreply.github.com>
Signed-off-by: Martin <81148675+Martian-0007@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Typo Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
- Fix typos in Installation - Add information to Installation - Improve readability of Usage Signed-off-by: Martin <81148675+Martian-0007@users.noreply.github.com>
- Remove redundant topics from Request response Signed-off-by: Martin <81148675+Martian-0007@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Martian <81148675+Martian-0007@users.noreply.github.com>
Signed-off-by: Martian <81148675+Martian-0007@users.noreply.github.com>
Signed-off-by: Martian <81148675+Martian-0007@users.noreply.github.com>
- Add Hardware Description (WIP) - Fix mistakes (ran through Grammarly) - Reformat with prettier Signed-off-by: Martian <81148675+Martian-0007@users.noreply.github.com>
a7d592c
to
5c5f509
Compare
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.
Pull Request Overview
Add comprehensive documentation for the new TAPPER product, including usage, installation, hardware specs, TLS setup, and API specification, and integrate it into the site sidebar and navigation.
- Introduce TAPPER documentation pages under
tapper/
- Update sidebar and Docusaurus configuration to include the TAPPER section
Reviewed Changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 4 comments.
Show a summary per file
File | Description |
---|---|
tapper/introduction.md | Added high-level introduction to TAPPER |
tapper/installation.md | Added step-by-step client installation guide |
tapper/hardware.md | Added hardware specifications and schematics |
tapper/tls-setup.md | Added MQTT over TLS setup instructions |
tapper/usage.md | Added CLI usage and configuration reference |
tapper/api-spec.md | Added MQTT API specification |
sidebars-tapper.js | Defined sidebar order for TAPPER docs |
docusaurus.config.js | Registered TAPPER docs plugin and added navbar item |
Comments suppressed due to low confidence (1)
docusaurus.config.js:107
- The
editUrl
line is misindented and placed outside the plugin object for the TAPPER docs—move it inside the({ ... })
block passed to@docusaurus/plugin-content-docs
.
editUrl: 'https://github.com/hardwario/docs/edit/main',
Signed-off-by: Martian <81148675+Martian-0007@users.noreply.github.com>
Signed-off-by: Martian <81148675+Martian-0007@users.noreply.github.com>
Signed-off-by: Martian <81148675+Martian-0007@users.noreply.github.com>
Signed-off-by: Martian <81148675+Martian-0007@users.noreply.github.com>
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.
Pull Request Overview
This PR adds a TAPPER section to the documentation covering client usage, TLS setup, API specifications, and hardware and installation guides.
- Added comprehensive CLI usage instructions for TAPPER in tapper/usage.md
- Provided detailed TLS setup and API specification guides in tapper/tls-setup.md and tapper/api-spec.md
- Updated supplementary documents (introduction, installation, hardware) and configuration files for integrated navigation
Reviewed Changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 2 comments.
Show a summary per file
File | Description |
---|---|
tapper/usage.md | New CLI usage documentation for TAPPER |
tapper/tls-setup.md | TLS certificate generation and Mosquitto setup guide |
tapper/introduction.md | Introduction and product overview for TAPPER |
tapper/installation.md | Instructions for installing and setting up the TAPPER client |
tapper/hardware.md | Hardware specifications and schematics for TAPPER |
tapper/api-spec.md | MQTT API specifications for TAPPER communication |
sidebars-tapper.js | Sidebar configuration for the new TAPPER documentation |
docusaurus.config.js | Updated configuration to include the TAPPER docs route |
Signed-off-by: Martian <81148675+Martian-0007@users.noreply.github.com>
Signed-off-by: Martian <81148675+Martian-0007@users.noreply.github.com>
Signed-off-by: Martian <81148675+Martian-0007@users.noreply.github.com>
Add a section for the TAPPER Product.