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

Signal Messanger - Add Markdown support #117554

Closed
GAS85 opened this issue May 16, 2024 · 5 comments
Closed

Signal Messanger - Add Markdown support #117554

GAS85 opened this issue May 16, 2024 · 5 comments

Comments

@GAS85
Copy link

GAS85 commented May 16, 2024

The problem

Hey, currently all messages will be send without markdown support, to enable it you need to add "text_mode": "styled" to the data.

What version of Home Assistant Core has the issue?

core-2024.5.1

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

signal_messenger

Link to integration documentation on our website

https://www.home-assistant.io/integrations/signal_messenger/

Diagnostics information

No response

Example YAML snippet

service: notify.signal
metadata: {}
data:
  message: >-
    🔋 *Battery* is -empty- 🪫

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

Hey there @bbernhard, mind taking a look at this issue as it has been labeled with an integration (signal_messenger) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of signal_messenger can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign signal_messenger Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


signal_messenger documentation
signal_messenger source
(message by IssueLinks)

@joostlek
Copy link
Member

Thank you for reaching out. We use GitHub for tracking issues, not for providing support or tracking feature requests.

If you want to suggest a feature, you should try our Community Forum: Feature Requests.

If you have additional questions, feel free to join our Discord chat server.

Thanks! 👍

@joostlek joostlek closed this as not planned Won't fix, can't repro, duplicate, stale May 16, 2024
@GAS85
Copy link
Author

GAS85 commented May 17, 2024

Just for info, it was already mentioned on the forum, but without any attention.

@bbernhard
Copy link
Contributor

bbernhard commented May 17, 2024

Unfortunately I do not have that much time right now to look into the HA integration (the last time I've contributed to the project, the plugin architecture was quite different and at the moment I am lacking time to read into that).

What I can offer you guys is, that we make the text_mode a global setting that can be set globally in the signal-cli-rest-api container. When set, all messages are sent with text_mode: styled. Would that be helful? If so, then please create an issue in the signal-cli-rest-api github repository.

@GAS85
Copy link
Author

GAS85 commented May 30, 2024

I suppose so, it is only 1 parameter to enable Markdown.

@github-actions github-actions bot locked and limited conversation to collaborators Jun 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants