Skip to content
Snippets Groups Projects

Refactor the node spy page

Merged Rick Parrish requested to merge Ree/node-spy-2 into master

Split the mqtt-related code out into a re-usable component, which will make it easier to use in other pages (eg the proof-of-concept web-based sbbsctrl), and allow the mqtt websocket ports to be entered via the website if they haven't been configured yet.

Split the javascript code out into separate .js files.

Rename the node spy with a 001 prefix to match the userlist prefix, so the "More" dropdown menu is alphabetized.

Merge request reports

Pipeline #4636 passed

Pipeline passed for 8b48288c on Ree/node-spy-2

Approval is optional

Merged by Rob SwindellRob Swindell 1 year ago (Aug 16, 2023 12:07am UTC)

Merge details

  • Changes merged into master with 0544e033 (commits were squashed).
  • Deleted the source branch.
  • Auto-merge enabled

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading