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

Mwpw-147034: Custom border color + badge/border color decoupling [Mer… #15

Merged
merged 8 commits into from
Jul 23, 2024

Conversation

Axelcureno
Copy link
Member

@Axelcureno Axelcureno commented Jun 24, 2024

Adds ability to decouple border and badge colors in authoring of merch card.
Adds ability to set custom border color without using a badge in merch card.

Resolves: MWPW-147034

Milo PR: adobecom/milo#2613

Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/axel/custom-border-badge?martech=off
After: https://mwpw-147034--milo--adobecom.hlx.page/drafts/axel/custom-border-badge?martech=off

@Axelcureno Axelcureno added enhancement New feature or request VALIDATED IN MILO Means QA has added the "Verified" label in Milo labels Jun 24, 2024
@Axelcureno Axelcureno self-assigned this Jun 24, 2024
Copy link

aem-code-sync bot commented Jun 24, 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

Copy link
Collaborator

@npeltier npeltier left a comment

Choose a reason for hiding this comment

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

lgtm

yesil
yesil previously requested changes Jun 27, 2024
web-components/src/merch-card.js Outdated Show resolved Hide resolved
Copy link

aem-code-sync bot commented Jul 22, 2024

Page Scores Audits Google
/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@3ch023 3ch023 removed the VALIDATED IN MILO Means QA has added the "Verified" label in Milo label Jul 23, 2024
@3ch023
Copy link
Collaborator

3ch023 commented Jul 23, 2024

UPD:
this PR wasn't yet consumed in MILO
Next steps:
milo PR with dependency & validation

@Roycethan Roycethan added the VALIDATED IN MILO Means QA has added the "Verified" label in Milo label Jul 23, 2024
@Axelcureno Axelcureno merged commit 339d1c5 into main Jul 23, 2024
4 of 5 checks passed
rohitsahu pushed a commit that referenced this pull request Jul 26, 2024
* main:
  MWPW-153245 [merch] make shadow elements spit custom events (#42)
  MWPW-148229  Left Rail filter hover Text  (#43)
  Stop using code which resolves default display tax label value per country and segment (#27)
  Mwpw-147034: Custom border color + badge/border color decoupling [Mer… (#15)
  MWPW-154026: Long CTAs fall in the second line in merch card footer (#38)

# Conflicts:
#	libs/commerce.js
#	libs/mas.js
#	libs/merch-card-all.js
#	libs/merch-card.js
#	web-components/src/merch-card.js
rohitsahu pushed a commit that referenced this pull request Aug 7, 2024
* 'ost' of https://github.com/adobecom/mas:
  Update mas.js
  Revert "resolved conflicts"
  Update mas.js
  updated deps
  MWPW-153599: M@S Studio solution design (#36)
  moved delay method
  Update package-lock.json
  Mwpw-151480: move of commerce and web components to milo (#49)
  [MWPW-154335] [callout] Spacing issue encountered when the call-out section is added (#45)
  MWPW-153245 [merch] make shadow elements spit custom events (#42)
  MWPW-148229  Left Rail filter hover Text  (#43)
  Stop using code which resolves default display tax label value per country and segment (#27)
  Mwpw-147034: Custom border color + badge/border color decoupling [Mer… (#15)
  MWPW-154026: Long CTAs fall in the second line in merch card footer (#38)

# Conflicts:
#	libs/commerce.js
#	libs/mas.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request VALIDATED IN MILO Means QA has added the "Verified" label in Milo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants