-
Notifications
You must be signed in to change notification settings - Fork 175
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
[Release] Stage to Main #3411
[Release] Stage to Main #3411
Conversation
add basic mep-actions test Co-authored-by: John Pratt <jpratt@adobe.com>
…r testing (#3147) * Redesign mini-compare chart. Added checkmarks for icons. Backward compatibility using "checkmark" class. Horizontal line added. * implemented cheveron up and down functionality for checkmark copy list * added isMobile check * Fix backward compatibility issues * minor * minor css fixes as per figma * refactored large "decorateFooterRows" method * remove duplicate css review comment * review comments * UT added * fix post merge conflict * npm run build * linter * UT coverage * mini compare covergae * review comment * fix post merge. margin was affected due to ul change * fixed ul after merge conflict with the other fix PR * linter * resolved conflicts * fixed mini-compare-chart footer cta alignment * decreased footer rows height to match the design * fixed mini-compare footer alignment * fixed footer horizontal padding and mobile breakpoint * updated unit tests * corrected paddings, font sizes and other * corrected slots * corrected padding * footer padding corrected * corrected heading-m-price padding --------- Co-authored-by: Rohit Sahu <rosahu@adobe.com> Co-authored-by: Mira <myrosvas@gmail.com>
* MWPW-163380 - Missing project details locui create * Added missing project details page for locui create as query params are lost after sucessfull login Resolves: https://jira.corp.adobe.com/browse/MWPW-163380 **Test URLs:** - Before: https://stage--milo--adobecom.hlx.page/drafts/sircar/locui-create?martech=off - After: https://mwpw-163380-instructions-loc-create--milo--saurabhsircar11.hlx.page/drafts/sircar/locui-create?martech=off * MWPW-163380 - Fixed some issues * Added margin zero * Fixed mock class Resolves: https://jira.corp.adobe.com/browse/MWPW-163380 **Test URLs:** - Before: https://stage--milo--adobecom.hlx.page/drafts/sircar/locui-create?martech=off - After: https://mwpw-163380-instructions-loc-create--milo--saurabhsircar11.hlx.page/drafts/sircar/locui-create?martech=off * feat: added links for respective environments * feat: changed text for goto list
* MWPW-163379: Fix edit url from Helix admin API for DA * MWPW-163379 : Addressing review comments * MWPW: updating with review comments * MWPW-163379: Implementing suggestions
* initial publish * first commit * cleaned up * fixed id dupes * tighten up accordion height, increase width * Add source * Remove shown * Fix freshManifest source * committing better version not neat * added divs to dropdowns * stab at data parse for db * refine parseMepConfig * align to db, improve form look, add event listeners * stop using block name constant * update to aws api * Unfinished code for Viv * Fix undefined manifest * Add back properties and fix small DOM issue * Add get getMepPopup * stash * save working * Confine listeners to current meplist * add url param and hash method for sharing * just omit promos out of date range * minor fixes and checks * add condition for mepHighlight * pulling in manifest info * don't show arrow on mep panel in mmm * refactor functions * Fix radio and preview buttons * Remove comment * Fix typo * rework share logic fix spacing color * fix merge conflict bug * fix merge conflict created bug * Add another pageId * Fix label click issue * Fix pencil issue * bunch of fixes * deep link * add conditional chaining * refactor and streamline * update now that pathname is sent by db * fix page search for new db structure * switch to full field name not first letter * change all to empty * top geos and top pages * remove line * fix breaking from merge * loops through filters instead of hard code * keep share button on right in mobile * no longer using form.html * default all doesn't need to be in Word * share button on top * create shareButton listeners at creation * rename functions * remove unneeded files * restore fix to update button * change served to found * add change listener to input * unit tests working * button back but not visible * button fixed * add sharebutton unit test * fix share button test * repair existing unit tests * update mmm test after other fixes * collapse all when new one opened * update to new db column names * fix issue with preview link including entire manifest url instead of just path * fix activites filter in saveToMmm * add source for mep param, fix filter for save * fix radio buttons to only use path * duplicate schedule info * mmmremovemepsource * fix page save and analyticsTitle * tweak placeholder helpful info * Fix code scanning alert no. 235: Incomplete URL substring sanitization Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com> * remove unused * tweak to suggested security fix * unit test update * curious what the security check says * revert * remove authorization * lint issue * simplify milo check * add source to default promo shell... again * update css rule for add manifest input * hide scheduled info in mmm * remove unused import and return error message * fix bugs found in stage * remove old call to save * move call to save back * MWPW-164118 [MMM] difference in promo start time * move manifest lengths check * convert to string if needed * adjust for date fix in mmm * unit test update * update to make code coverage easier * add ignore for catch --------- Co-authored-by: viloria <viloria@adobe.com> Co-authored-by: markpadbe <markp@adobe.com> Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Testing can start @adobecom/bacom-sot @adobecom/creative-cloud-sot @adobecom/document-cloud-sot @adobecom/express-sot @adobecom/homepage-sot @adobecom/miq-sot |
Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit
eslint
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3411 +/- ##
==========================================
- Coverage 96.48% 96.47% -0.01%
==========================================
Files 254 255 +1
Lines 59031 59253 +222
==========================================
+ Hits 56954 57163 +209
- Misses 2077 2090 +13 ☔ View full report in Codecov by Sentry. |
* chore: ignore tmp * Move title-append logic to utils/loadArea * mark existing tests * delete existing code * Fix tests * nits * different pr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
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: