Remove warning about emails being lost #487

Merged

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

snedadah commented 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.

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. 🙂

a258wang merged commit 8c43ed47b2 into main 2 months ago

a258wang deleted branch shahanneda/remove-warning 2 months ago

a258wang referenced this issue from a commit 2 months ago

Remove warning about emails being lost (#487)

Reviewers

r389li was requested for review 2 months ago

a258wang approved these changes 2 months ago

continuous-integration/drone/push Build is passing

Details

The pull request has been merged as 8c43ed47b2.

Sign in to join this conversation.

No Label

No Milestone

No project

No Assignees

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: www/www-new#487