Remove warning about emails being lost #487

Merged

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

snedadah commented 6 months ago

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.

snedadah added 1 commit 6 months ago

942ea1ee06

Removed warning

snedadah requested review from r389li 6 months ago

snedadah requested review from a258wang 6 months ago

a258wang approved these changes 6 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 6 months ago

a258wang deleted branch shahanneda/remove-warning 6 months ago

a258wang referenced this issue from a commit 6 months ago

Remove warning about emails being lost (#487)

Reviewers

r389li was requested for review 6 months ago

a258wang approved these changes 6 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