Skip to content

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

Open
wants to merge 23 commits into
base: main
Choose a base branch
from
Open

Add TAPPER section #141

wants to merge 23 commits into from

Conversation

Martian-0007
Copy link
Contributor

Add a section for the TAPPER Product.

Copy link

netlify bot commented May 11, 2025

Deploy Preview for hardwario-docs ready!

Name Link
🔨 Latest commit faf3be8
🔍 Latest deploy log https://app.netlify.com/projects/hardwario-docs/deploys/683dbe95583bdc0008c9ca98
😎 Deploy Preview https://deploy-preview-141--hardwario-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Martian-0007 Martian-0007 requested a review from Copilot May 26, 2025 09:10
@Martian-0007 Martian-0007 marked this pull request as ready for review May 26, 2025 09:10
Copy link

@Copilot Copilot AI left a 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.

@Martian-0007
Copy link
Contributor Author

The checks failed on initialization - failed to enqueue.
Ideally, could we re-run the checks?

Copy link

@Copilot Copilot AI left a 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

@Martian-0007 Martian-0007 requested a review from Copilot May 26, 2025 14:19
Copy link

@Copilot Copilot AI left a 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'),

Martian-0007 and others added 16 commits May 31, 2025 22:35
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>
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>
Copy link

@Copilot Copilot AI left a 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>
@Martian-0007 Martian-0007 requested a review from Copilot May 31, 2025 20:59
Copy link

@Copilot Copilot AI left a 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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant