-
Notifications
You must be signed in to change notification settings - Fork 18
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Added documentation on new test long code and email (#1941)
- Loading branch information
1 parent
4ae7046
commit 879b936
Showing
1 changed file
with
15 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -173,6 +173,21 @@ Jinja templates are pulled in from the [notification-utils](https://github.com/c | |
|
||
6. Remove `USE_LOCAL_JINJA_TEMPLATES=True` from your .env file, and delete any jinja in `jinja_templates`. Deleting the folder and jinja files is not required, but recommended. Make sure you're pulling up-to-date jinja from notification-utils the next time you need to make changes. | ||
|
||
## Testing | ||
|
||
To help debug full code paths of emails and SMS, we have a special email and phone number | ||
set in the application's configuration. As it stands at the moment these are the following: | ||
|
||
| Notification Type | Test destination | | ||
|-------------------|--------------------------| | ||
| Email | [email protected] | | ||
| SMS | +16135550123 | | ||
|
||
Whereas the smoke test emails and long codes might not get through the whole GCNotify | ||
data treatment, these will and have proper database fields populated. This is useful | ||
for proper stress tests where the notifications shouldn't merely touch the API | ||
front-door but also get through the Celery workers processing. | ||
|
||
## Frequent problems | ||
|
||
__Problem__: No *postgres* role exists. | ||
|