Remove warning about emails being lost #487

Merged

a258wang merged 1 commits from shahanneda/remove-warning into main

2022-08-03 23:06:44 -04:00

snedadah commented

2022-08-03 21:09:03 -04:00

Owner

I don't think we should keep a long message in the file, since all the warnings are loaded on every page load, so it might have a negative effect on performance over a long period if we keep all warnings.

Feel free to merge on green.

I don't think we should keep a long message in the file, since all the warnings are loaded on every page load, so it might have a negative effect on performance over a long period if we keep all warnings. Feel free to merge on green.

snedadah added 1 commit 2022-08-03 21:09:05 -04:00

continuous-integration/drone/push Build is passing Details

942ea1ee06

Removed warning

snedadah requested review from r389li 2022-08-03 21:11:39 -04:00

snedadah requested review from a258wang 2022-08-03 21:11:40 -04:00

a258wang approved these changes 2022-08-03 23:06:31 -04:00

a258wang left a comment

Owner

The string is short enough that it isn't going to have a noticeable impact on latency or anything. But this works too. 🙂

The string is short enough that it isn't going to have a noticeable impact on latency or anything. But this works too. 🙂

a258wang merged commit 8c43ed47b2 into main

2022-08-03 23:06:44 -04:00

a258wang deleted branch shahanneda/remove-warning

2022-08-03 23:06:45 -04:00

a258wang referenced this issue from a commit

2022-08-03 23:06:46 -04:00

Remove warning about emails being lost (#487)

Sign in to join this conversation.

No reviewers

No Label

Feature Request

Good First Issue

Priority - High

Priority - Medium

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: www/www-new#487

No description provided.