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

feat(notification): facebook workplace integration #219

Merged
merged 2 commits into from
Jun 11, 2021

Conversation

adeyahya
Copy link
Contributor

@adeyahya adeyahya commented Jun 10, 2021

Hello everyone, I made this pr so that Monika can send notifications via Facebook Workplace. Please kindly check and review it thanks.

| ---- | ----------------------------------------------- | ------------------ |
| ID | Notification identity number | `Workplace12345` |
| Type | Notification types | `slack` |
| ID | It's located at thread url, in the last segment | `6367478493277649` |
Copy link
Member

Choose a reason for hiding this comment

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

I think the key here should be thread_id, right? Can you also please add an example of the thread URL?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ah ya, it should be thread_id, I will update it. and here is the thread URL example

https://workwork.workplace.com/chat/t/6367478493277649

@dennypradipta
Copy link
Contributor

Hello @adeyahya thank you for contributing!

May I ask you to provide screenshots of the notifications sent to Facebook Workplace? 😀
You should probably censor the sensitive data if there is any.

Thank you

Copy link
Contributor

@kevin-hyperjump kevin-hyperjump left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks for the PR @adeyahya 🍻

@adeyahya
Copy link
Contributor Author

Hello @adeyahya thank you for contributing!

May I ask you to provide screenshots of the notifications sent to Facebook Workplace? 😀
You should probably censor the sensitive data if there is any.

Thank you

here is the screenshot
Screen Shot 2021-06-11 at 16 55 02

@adeyahya adeyahya force-pushed the main branch 2 times, most recently from bc5cc98 to 78121c5 Compare June 11, 2021 10:05
@haricnugraha haricnugraha merged commit 8c22a4b into hyperjumptech:main Jun 11, 2021
kevin-hyperjump pushed a commit to kevin-hyperjump/monika that referenced this pull request Aug 13, 2021
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.

6 participants