Skip to content
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

[jsk_robot_startup] Update Readme; configuring a smtp server #1581

Merged
merged 4 commits into from
Sep 9, 2022

Conversation

tkmtnt7000
Copy link
Member

I have written a readme about the work that needs to be done before using email_topic.py.
We have to configure a smtp server to use email_topic.py

If there is a better position to put this, please tell me.

@@ -10,7 +10,7 @@ see [lifelog/README.md](lifelog/README.md)
This node sends email based on received rostopic (jsk_robot_startup/Email).
Default values can be set by using `~email_info`
There is [a client library](./euslisp/email-topic-client.l) and [sample program](./euslisp/sample-email-topic-client.l).
If you want to see a demo. Please configure a smtp server and setup your email_info yaml at /var/lib/robot/email_info.yaml and run.
If you want to see a demo. Please [configure a smtp server](https://github.com/tkmtnt7000/jsk_robot/blob/PR-update-switchbot-device-name/jsk_robot_common/jsk_robot_startup/README.md#configuring-a-smtp-server-with-gmail) and setup your email_info yaml at /var/lib/robot/email_info.yaml and run.
Copy link
Member

Choose a reason for hiding this comment

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

please update github link to https://github.com/jsk-ros-pkg/jsk_robot becuase https://github.com/tkmtnt7000/jsk_robot does not work if you delete branch.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you. I fixed link in 63cead7

@k-okada k-okada merged commit bf34d95 into jsk-ros-pkg:master Sep 9, 2022
@tkmtnt7000 tkmtnt7000 deleted the PR-update-readme-email-topic branch June 6, 2023 01:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants