Simplify feature accuracy notification email

[KyleJu]

I have a few ideas in #4355 (comment). Overall, the idea is to simplify the sentences and reduce the overhead of reading it. Instead highlight the actions on top.

Remove use of extra []s and change wording in email subjects:

• From [webstatus] [Action requested] Update Saved queries in sharedStorage.selectURL to [webstatus] Action requested - Verify Saved queries in sharedStorage.selectURL
• When escalation, from [webstatus] ESCALATED: [Action requested] Update Saved queries in sharedStorage.selectURL to [webstatus] Escalation request - Verify Saved queries in sharedStorage.selectURL

Rearrange the email body:

• Move Verify feature accuracy button to the top
• Merge four sections into three
• Highlight the feature is launched soon in the first sentence

Regular accuracy notification (Note that the email subject in prod will start with [webstatus]):

Escalation email:

Before this change:

Other ideas

On top of it, wdyt if I grey the section under id=why-triggered? Normally the explanation for "why I receive this email" is always greyed in the email

[DanielRyanSmith]

1. Where does the [webstatus] prefix come from in the email subject?

I'm not certain here, but this could be a byproduct of [email protected] Google group sending these emails to our inbox. E.g. None of the actual emails have the prefix in the group

2. Is there a way to update test_build_email_tasks_escalated_feature_accuracy.html test automatically?

Yes - uncommenting the line here and then running the test will update the contents of the file that is being compared (be sure to re-comment the line back so it's not trying to rewrite the contents each time)

[past]

This is much improved, thanks! I didn't look at the code too closely as I assume Jason will, but I have one more nit: can we refer to the "Chromestatus team" instead of the "Webstatus team" everywhere, including the email subject if possible?

Nowadays with webstatus.dev being a separate thing, the confusion becomes a bit more concerning.

[jrobbins]

On top of it, wdyt if I grey the section under id=why-triggered?

I'd prefer not to use gray text. The fact that it comes last should be enough.

[jrobbins]

LGTM after one nit.