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

[Release] Stage to Main #3411

Merged
merged 10 commits into from
Jan 7, 2025
Merged

[Release] Stage to Main #3411

merged 10 commits into from
Jan 7, 2025

Conversation

jpratt2 and others added 7 commits January 2, 2025 09:16
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>
@milo-pr-merge
Copy link
Contributor Author

milo-pr-merge bot commented Jan 2, 2025

Testing can start @adobecom/bacom-sot @adobecom/creative-cloud-sot @adobecom/document-cloud-sot @adobecom/express-sot @adobecom/homepage-sot @adobecom/miq-sot

@milo-pr-merge milo-pr-merge bot requested a review from a team as a code owner January 2, 2025 09:16
Copy link
Contributor

aem-code-sync bot commented Jan 2, 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
Contributor

aem-code-sync bot commented Jan 2, 2025

Page Scores Audits Google
📱 /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

Copy link
Contributor

@github-actions github-actions bot left a 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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/f4721809bfef837bb7c4e462074569e6fc41dc4b/libs/deps/mas/merch-subscription-panel.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/f4721809bfef837bb7c4e462074569e6fc41dc4b/libs/deps/mas/merch-twp-d2p.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/f4721809bfef837bb7c4e462074569e6fc41dc4b/libs/deps/mas/merch-whats-included.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/f4721809bfef837bb7c4e462074569e6fc41dc4b/libs/features/mas/dist/mas.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/f4721809bfef837bb7c4e462074569e6fc41dc4b/libs/features/mas/src/global.css.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/f4721809bfef837bb7c4e462074569e6fc41dc4b/libs/features/mas/src/variants/mini-compare-chart.css.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/f4721809bfef837bb7c4e462074569e6fc41dc4b/libs/features/mas/src/variants/mini-compare-chart.js

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link

codecov bot commented Jan 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.47%. Comparing base (d543279) to head (a2443c0).
Report is 16 commits behind head on main.

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.
📢 Have feedback on the report? Share it here.

* chore: ignore tmp

* Move title-append logic to utils/loadArea

* mark existing tests

* delete existing code

* Fix tests

* nits

* different pr
@milo-pr-merge milo-pr-merge bot requested a review from hparra as a code owner January 2, 2025 12:19
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

@milo-pr-merge milo-pr-merge bot merged commit 55c4a2d into main Jan 7, 2025
77 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.