Repository navigation
Conversation
The Synapse email templates load web fonts through stylesheets in their <head>. The mail client puts the email in an iframe, and the email text is in the DOM before those stylesheets load, but the browser paints nothing in the frame until they do. The helpers checked only the text, so a slow stylesheet gave a screenshot of a blank pane. In one failure the Poppins 700 stylesheet had no response when the screenshot was taken. validateEmail and validateEmailForResetPassword now wait until the email's stylesheets, images and fonts have loaded. A timeout lists what is still pending, and a wait of more than a second is logged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WdM45xnz3vvTmA7HN8nGmK
…esheet Holds the Poppins 700 stylesheet the email templates load for 8 seconds and runs only the three email screenshot tests on each matrix shard, repeated, one worker, no retries. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WdM45xnz3vvTmA7HN8nGmK
Contributor
Author
|
[Claude Code 🤖] Verification finished; results are summarised in #6883. Closing this throwaway branch. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Warning
Do not merge. This is a throwaway branch that checks a flaky-test fix. It will be closed after CI runs.
The Poppins 700 stylesheet that the Synapse email templates load is held for 8 seconds. In a CI failure of
profile.spec.ts › it can change email in settings, that stylesheet had no response when the screenshot was taken, and the email showed as a blank pane. Each matrix shard runs only the three email screenshot tests, 5 times each, one worker, no retries.This branch is the wait-for-render fix (buck-25-wait-for-email-render). Expected: pass, with a logged wait of about 8 seconds.
🤖 Generated with Claude Code
https://claude.ai/code/session_01WdM45xnz3vvTmA7HN8nGmK