Skip to content

[OPT-39999] Mep-xlg-tags.json issues#5482

Open
AdobeLinhart wants to merge 3 commits intostagefrom
OPT-39999
Open

[OPT-39999] Mep-xlg-tags.json issues#5482
AdobeLinhart wants to merge 3 commits intostagefrom
OPT-39999

Conversation

@AdobeLinhart
Copy link
Contributor

@AdobeLinhart AdobeLinhart commented Mar 3, 2026

Adds a check to prevent getXLGListURL() from firing multiple times by utilizing promises. It should only perform the call once.

Behavior

  • Do not execute network call for mep-xlg-tags if signed out.
  • Execute a single network call for mep-xlg-tags if signed in.

Instructions

  • Open test URL
  • Open the network panel
  • Filter for mep-xlg-tags
  • Observe the amount of times called
  • Test both as a logged in and logged out user.

Before:
image

After:
image

Resolves: OPT-39999

Test URLs:

@AdobeLinhart AdobeLinhart requested a review from a team as a code owner March 3, 2026 00:56
@aem-code-sync
Copy link
Contributor

aem-code-sync bot commented Mar 3, 2026

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

  • Re-sync branch
Commits

Copy link
Contributor

@jpratt2 jpratt2 left a comment

Choose a reason for hiding this comment

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

Approved.
Verified: Only one call is seen in the network tab when logged in. (see attached) The content of the network call appears unharmed.

Image

Verified: there is no network call when logged out

@AdobeLinhart AdobeLinhart requested a review from a team March 6, 2026 22:35
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.

2 participants