Skip to content

Ssr flag update - #99

Open
JasonHowellSlavin wants to merge 4 commits into
stagefrom
ssr-flag-update
Open

Ssr flag update#99
JasonHowellSlavin wants to merge 4 commits into
stagefrom
ssr-flag-update

Conversation

@JasonHowellSlavin

Copy link
Copy Markdown
Collaborator

Description

  • Changes the variable name from ssrFlag to headAlreadyLoaded
  • Removes optional chaining since we load the html directly in the head

Related Issue

Resolves: MWPW-188278

Test URLs

@aem-code-sync

aem-code-sync Bot commented Apr 28, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

@aem-code-sync

aem-code-sync Bot commented Apr 28, 2026

Copy link
Copy Markdown
Page Scores Audits
📱 /acrobat/online/compress-pdf PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS
🖥️ /acrobat/online/compress-pdf PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS

@codecov-commenter

codecov-commenter commented Apr 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.29%. Comparing base (e222498) to head (c914724).
⚠️ Report is 48 commits behind head on stage.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage      #99      +/-   ##
==========================================
+ Coverage   74.28%   74.29%   +0.01%     
==========================================
  Files          45       45              
  Lines        9593     9593              
==========================================
+ Hits         7126     7127       +1     
+ Misses       2467     2466       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Ruchika4

Copy link
Copy Markdown
Collaborator

Hi @JasonHowellSlavin , thanks for the PR. We will need the SHA-250 also updated for this change to reflect in our frictionless pages where the edgeworkers are enabled to do pre render for performance reasons. Here is a sample PR for references where SHA is updated #83 Could you please add that update also as part of this PR so that you changes reflect on DC pages? Please let me know if any questions.

@aem-code-sync
aem-code-sync Bot temporarily deployed to ssr-flag-update May 6, 2026 17:25 Inactive
@thedoc31

Copy link
Copy Markdown

No activity in 7 days. Will be labeled stale at 14 days. Auto-close is disabled this run. Add stale-ignore to exempt.

No activity in 14 days — labeled stale. Auto-close is disabled this run. Add stale-ignore to exempt.

No activity in 21 days — labeled autoclose-pending. Would normally close in 7 days, but auto-close is disabled this run. Add autoclose-ignore to exempt.

No activity in 28 days. Would normally close now, but auto-close is disabled this run. Add autoclose-ignore to exempt.


Live rollout: adobecom+Adobe-acom 10-repo sample, run by casalino via Claude Code

@adobecom-git-stale-pr-bot adobecom-git-stale-pr-bot Bot removed stale No recent activity autoclose-pending Will be auto-closed soon labels Sep 3, 2026
@adobecom-git-stale-pr-bot

Copy link
Copy Markdown

No activity in 7 days. Will be labeled stale at 14 days. Auto-close is disabled this run. Add stale-ignore to exempt.


Rundeck job Git Stale PR Check (git-stale-pr-bot), execution 2739336, project global, run by casalino

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants