Skip to content

Rohan/slackbot#6

Open
rohanbhadange wants to merge 2 commits into
mainfrom
rohan/slackbot
Open

Rohan/slackbot#6
rohanbhadange wants to merge 2 commits into
mainfrom
rohan/slackbot

Conversation

@rohanbhadange

Copy link
Copy Markdown
Collaborator

Summary

Adds Slack-based remote communication helpers for ScoutMini, including a ROS
gateway, Slack command bot, status publishing utilities, and documentation for
operating the setup safely.

The goal is to let the robot report status and accept limited remote commands
through Slack while keeping credentials external to the repository.

Scope

  • Adds remote communication documentation under docs/remote_communication/
  • Adds SendSlackMessage.srv to scoutmini_interfaces
  • Adds Slack gateway, command bot, and status publisher scripts to
    scoutmini_tasks
  • Adds helper scripts under scripts/slack/
  • Adds placeholder .env examples without real credentials

@rohanbhadange rohanbhadange requested a review from nhat-m-le as a code owner July 9, 2026 19:02
@nhat-m-le nhat-m-le self-assigned this Jul 9, 2026

@nhat-m-le nhat-m-le left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First round of review. Two main things: create a scoutmini_slack package and put the changes in scoutmini_tasks over there; need some placeholder callback to handle incoming message.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy the content to the actual PR and remove this file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See what's still needed to keep in this file otherwise remove

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revise to see what to keep

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be handled by repo-wide .gitignore

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this handle receiving message yet? I'd want to be able to handle incoming message knowing where they come from and on which channel and the timestamp

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's create a scoutmini_slack and put all the changes in this scoutmini_tasks package for Slack over there.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be handled by repo-wide .gitignore instead

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.

2 participants