Skip to content

Conversation

tuanphan08
Copy link
Collaborator

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):

Fix #

Test URLs:

Tuan Phan added 3 commits June 19, 2025 23:08
…s. Temporary remove tools/jira-ui/test-auth.js due to existence of apiToken which violates GitHub policy
Copy link

aem-code-sync bot commented Jun 20, 2025

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

Copy link

aem-code-sync bot commented Jun 20, 2025

Page Scores Audits
📱 / PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS
🖥️ / PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS

@tuanphan08
Copy link
Collaborator Author

Lint errors are not related to this PR changes:

/home/runner/work/inside-aem/inside-aem/blocks/featured-articles/featured-articles.js
Error:   14:3   error  iterators/generators require regenerator-runtime, which is too heavyweight for this guide to allow them. Separately, loops should be avoided in favor of array iterations  no-restricted-syntax
Error:   15:21  error  Unexpected `await` inside a loop                                                                                                                                           no-await-in-loop
Error:   40:2   error  Newline required at end of file but not found                                                                                                                              eol-last

/home/runner/work/inside-aem/inside-aem/blocks/tags/tags.js
Error:   6:9  error  'target' is assigned a value but never used  no-unused-vars

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.

1 participant