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

email alerts #51

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

email alerts #51

wants to merge 4 commits into from

Conversation

emilyliublair
Copy link
Contributor

Description

Closes (#43)

Type of change

πŸ€• Bug Fix
πŸ†• Feature
πŸ” Chore/Cleanup
✨ Refactor

How to Test

  1. comment to add notification objects
  2. verify that notification objects are not created for users with no notification preference
  3. netlify dev
  4. netlify functions:invoke notif to send email

Checklist

  • Code follows design and style guidelines
  • Code is commented with doc blocks
  • Latest code has been rebased from base branch (usually develop)
  • Commits follow guidelines (concise, squashed, etc)
  • Github issues have been linked in relevant commits
  • Relevant reviewers (EM) have been assigned to this PR

Copy link

netlify bot commented Nov 17, 2024

βœ… Deploy Preview for focus-ga ready!

Name Link
πŸ”¨ Latest commit 7ff66db
πŸ” Latest deploy log https://app.netlify.com/sites/focus-ga/deploys/673a8196d87fd9000811fe1f
😎 Deploy Preview https://deploy-preview-51--focus-ga.netlify.app
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@emilyliublair emilyliublair marked this pull request as ready for review November 17, 2024 23:53
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.

3 participants