Skip to content

ACCS-873 - Enhance pay page with titles - #1306

Open
pablomoreno61 wants to merge 6 commits into
pay-by-linkfrom
ACCS-873-enhance-pay-page
Open

ACCS-873 - Enhance pay page with titles#1306
pablomoreno61 wants to merge 6 commits into
pay-by-linkfrom
ACCS-873-enhance-pay-page

Conversation

@pablomoreno61

@pablomoreno61 pablomoreno61 commented Jun 22, 2026

Copy link
Copy Markdown

@aem-code-sync

aem-code-sync Bot commented Jun 22, 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 all PSI checks
  • Re-run failed PSI checks
  • Re-sync branch
Commits

@github-actions

github-actions Bot commented Jun 22, 2026

Copy link
Copy Markdown

📝 Block Changes Detected

This PR contains changes to the following blocks:

  • blocks/commerce-pay-by-link/ (has README.md)

📋 README Requirements

Please ensure the README.md for each changed block includes:

  • Overview: Clear description of the block's purpose and functionality
  • Configuration: All configuration options, their types, defaults, and side effects
  • Integration: URL parameters, localStorage usage, and event handling
  • Behavior Patterns: User interaction flows and page context detection
  • Error Handling: How the block handles errors and fallback scenarios

✅ All blocks have README files

Please review and update the README files to ensure they accurately reflect any changes made to the blocks.


This comment was automatically generated by the Block README Check workflow.

@aem-code-sync

aem-code-sync Bot commented Jun 22, 2026

Copy link
Copy Markdown
Page Scores Audits Google
📱 /pay PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ /pay PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync
aem-code-sync Bot temporarily deployed to ACCS-873-enhance-pay-page June 22, 2026 12:45 Inactive
@aem-code-sync
aem-code-sync Bot temporarily deployed to ACCS-873-enhance-pay-page June 22, 2026 12:47 Inactive
@aem-code-sync
aem-code-sync Bot temporarily deployed to ACCS-873-enhance-pay-page June 22, 2026 12:52 Inactive
@aem-code-sync
aem-code-sync Bot temporarily deployed to ACCS-873-enhance-pay-page June 22, 2026 12:56 Inactive
@spittfree

Copy link
Copy Markdown

Test comment

letter-spacing: var(--type-body-1-default-letter-spacing);
color: var(--color-neutral-700);
font: var(--type-headline-3-strong-font);
letter-spacing: var(--type-headline-3-strong-letter-spacing);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test coment 2

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test

@JacobBrownAustin

Copy link
Copy Markdown

test

div.append(dt, dd);
dl.append(div);
});
totalsSection.append(dl);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are we no longer adding dl to the totalsSection ?

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.

3 participants