Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stage clone #7

Merged
merged 2 commits into from
Mar 6, 2024
Merged

Stage clone #7

merged 2 commits into from
Mar 6, 2024

Conversation

mokimo
Copy link
Owner

@mokimo mokimo commented Mar 6, 2024

  • Add your
  • Specific
  • Features or fixes

Resolves: MWPW-NUMBER

Test URLs:

Copy link

aem-code-sync bot commented Mar 6, 2024

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

@mokimo mokimo merged commit 45e00e7 into main-clone Mar 6, 2024
10 of 12 checks passed
@mokimo mokimo deleted the stage-clone branch March 6, 2024 14:46
@mokimo mokimo restored the stage-clone branch March 6, 2024 14:46
mokimo pushed a commit that referenced this pull request Dec 6, 2024
adobecom#3258)

* move common snippets to helper, add defer logic and make interact call the 1st thing in the flow for signed out users

* toggle FPID and ECID logic, re-organise exports, dont fire if consent cookie present

* Pre merge branch (#5)

move methods utils , use async load of helper, review comments

* move try catch block as per promise

* Pre merge branch (#6)

* Test remove martech logic (#7)

fix delay logic, fix pr comments

* Test remove martech logic (#8)

change uuid logic, promise structure, alloy response logic, requestid,

* Final local branch (#9)

fix review comments, add test cases

* fix coverage for if check

* fix coverage errors

* add test case for martech helpers

* send full cookie flag added post testing round 1
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