Skip to content
This repository has been archived by the owner on Oct 18, 2023. It is now read-only.

[REQUEST] Reminder notifications should be sent via Slack, not via email #108

Open
2 tasks done
etagwerker opened this issue Jun 3, 2022 · 0 comments
Open
2 tasks done
Labels
enhancement good first issue Good to get started with your contributions

Comments

@etagwerker
Copy link
Member

Before we start...:

  • I checked the documentation and didn't find this feature
  • I checked to make sure that this feature has not already been requested

Branch/Commit:

main branch.

Describe the feature:

As a Noko user
I forget to submit my time entries
And then I get a Slack notification
And I get the opportunity to make it right

Problem:

Email notifications can be easily ignored. Slack notifications not so much.

Resources:

Smart notifications (#98) introduced an integration with Slack. I believe we should go ahead and use that.

Email notifications are over here:

https://github.com/fastruby/pecas/blob/main/app/models/user.rb#L8-L16

And here:

https://github.com/fastruby/pecas/blob/main/app/mailers/reminder.rb#L4-L9

I will abide by the code of conduct

@etagwerker etagwerker added enhancement good first issue Good to get started with your contributions labels Jun 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement good first issue Good to get started with your contributions
Projects
None yet
Development

No branches or pull requests

1 participant