-
Notifications
You must be signed in to change notification settings - Fork 706
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
adding mongodbnotify to community #2505
Conversation
Next StepsHello! Thank you so much for your change 🤜 🤛 . There are a few things you need to do:
Manual Review RequiredHang tight! A Tidbyt engineer will be by shortly to review your change. Here is what they will be looking for:
|
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
Any movement on this? I'll be happy to hop on a call and explain / demonstrate, if it helps. |
apps/mongodbnotify/manifest.yaml
Outdated
id: mongodb-notify | ||
name: MongoDB Notify | ||
summary: Message notifications | ||
desc: Uses MongoDB for notifications. Can be used with Home Assistant to get Notifications to Tidbyt. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please update the app description a bit? Would be good if it explained that users will likely have to go read the README here in the community repo to get this set up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, @matslina,
I've updated the desc as you requested.
This uses an Atlas-MongoDB cluster (M0-free) to allow for notifications to be sent from anywhere but I mainly built it to allow Tidbyt to display notifications from Home Assistant.