You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Not every Sapling instance will necessarily need to send emails (or in the future, texts or push notifications), so making every Sapling instance install nodemailer and front-matter for that seems unnecessary.
Refactor notifications to be their own separate dependency that can be optionally installed to Sapling.
The text was updated successfully, but these errors were encountered:
Open question remains on how should the built-in password reset hook function if the Notification dependency is not present - although given that it can similarly be currently nonfunctional if nodemailer hasn't been configured correctly, this may be a wider consideration.
Not every Sapling instance will necessarily need to send emails (or in the future, texts or push notifications), so making every Sapling instance install
nodemailer
andfront-matter
for that seems unnecessary.Refactor notifications to be their own separate dependency that can be optionally installed to Sapling.
The text was updated successfully, but these errors were encountered: