Remove warning about emails being lost #487

Merged
a258wang merged 1 commits from shahanneda/remove-warning into main 2 months ago
Collaborator

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 2 months ago
942ea1ee06
Removed warning
snedadah requested review from r389li 2 months ago
snedadah requested review from a258wang 2 months ago
a258wang approved these changes 2 months ago
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 2 months ago
a258wang deleted branch shahanneda/remove-warning 2 months ago

Reviewers

r389li was requested for review 2 months ago
a258wang approved these changes 2 months ago
continuous-integration/drone/push Build is passing
The pull request has been merged as 8c43ed47b2.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: www/www-new#487
Loading…
There is no content yet.