[Release] Stage to Main #2479
Merged
[Release] Stage to Main #2479
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![milo-pr-merge[bot]](https://avatars.githubusercontent.com/u/95242840?s=60&v=4){kind=small}
…2229) * MWPW-146856: Left/right padding missing on segment cards on mobile Fixes Left/right padding missing on segment cards on mobile viewport. Tacocat PR: https://git.corp.adobe.com/wcms/tacocat.js/pull/584 Resolves: MWPW-146856 Test URLs: Before: https://main--milo--adobecom.hlx.page/?martech=off After: https://MWPW-146856--milo--adobecom.hlx.page/?martech=off * Update merch-card.js * Update merch-card.js * Update merch-card.js * updated dependency * Update merch-card.js
* modified georouting and pep so that pep only appears if a) GRM is not shown or b) GRM is dismissed * Unit test for GRM/PEP interaction * changed grmActive to geoRoutinActive * Fixed unit test for grm/pep interaction * removed unused imports * Cleaned up the implementation * Changed from polling to listening for an event * removed an unnecessary line * Modified unit test * Changed the implementation to use polling for any modal * Changed pep loading logic to poll for modals only after the close modal dialog comes through
* MWPW-147206 making promo schedule geo aware mostly to allow authors enabling promos in separate smaller files, schedule directive has now a locales setting to narrow its activation to those countries promoEnabled parameter passed to getPromoManifests in promo utils is now an object that has entries for each region promo name that are fetched at very early time (no geo resolution), default being kept for backward compatibility reasons with key `manifestnames`. getPromoManifests then resolves current locale and serves merged manifests between regional specific ones (if locale matches), and 'global' ones. * lint & ut fix * MWPW-147206 removing non matching manifests * lint fix * review fixes * MWPW-147206 rares changes
* Removing data layer descriptions from fields * removing console.log
|
Testing can start @adobecom/miq-sot @adobecom/bacom-sot @adobecom/homepage-sot @adobecom/creative-cloud-sot @adobecom/document-cloud-sot |
milo-pr-merge
bot
requested review from
Brandon32,
meganthecoder,
JasonHowellSlavin and
a team
as code owners
|
Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
|
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2479 +/- ##
==========================================
+ Coverage 95.66% 95.71% +0.04%
==========================================
Files 173 174 +1
Lines 45397 45641 +244
==========================================
+ Hits 43431 43686 +255
+ Misses 1966 1955 -11 ☔ View full report in Codecov by Sentry. |
* [MWPW-151172] Package updates * [MWPW-151172] Update dependencies; add zero-impact file
mwpw-152624: improve stage communication
* Mwpw-136871: TwP merch card (#2088) * MWPW-136871: TwP Merch Card * unit tests * TWP block draft (#2105) * add draft for a merch-twp block --------- Co-authored-by: Axel Cureno Basurto <[email protected]> * Mwpw 136871: Price display inside TwP merch card (#2110) * MWPW-136871: TwP Merch Card * unit tests * twp card * price display in TwP * deps update * Mwpw 136871 (#2154) * MWPW-136871: TwP Merch Card * unit tests * twp card * price display in TwP * deps update * MWPW-138927: merch-twp WIP (#2160) Merging proactively for a GWP demo. * MWPW-138927: desktop step 2 layout WIP * remove comment * pr feedback * update deps * update deps * update deps * update deps * fixed tests * fix lagging subscription panel * WIP * fix tests * fix tests * wip * fix failing test * fix failing tests * remove files * improve code coverage * address first batch of comments * fix dependencies * fix unit tests * add missing test cases --------- Co-authored-by: Axel Cureno Basurto <[email protected]> Co-authored-by: Mariia Lukianets <[email protected]> Co-authored-by: Axel Cureno Basurto <[email protected]>
Roycethan
approved these changes
Jun 17, 2024
JackySun9
approved these changes
Jun 17, 2024
TsayAdobe
approved these changes
Jun 17, 2024
sigadamvenkata
temporarily deployed
to
milo_pr_merge
GitHub Actions
Inactive
sigadamvenkata
approved these changes
Jun 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
common base root URLs
Homepage : https://www.stage.adobe.com/
BACOM: https://business.stage.adobe.com/fr/
CC: https://www.stage.adobe.com/creativecloud.html
Blog: https://blog.stage.adobe.com/
Acrobat: https://www.stage.adobe.com/acrobat/online/sign-pdf.html
Milo: