MWPW-155425: Caas Authoring - Ability to select multiple Event states #3224
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
…nd individual SVG assets (adobecom#2986)" This reverts commit f379815.
…nd individual SVG assets (adobecom#3182)
* [MWPW-160011] - GNAV light mode color changes (adobecom#3019) * Color changes * Promo background color fix * Add text and subtext font color changes in megamenu * cross cloud menu background color changes * Fix reveiew comments * colors hexcode formatted as lowercase for css variables * Bug fix MWPW-161889 * MWPW-161590, 161606, 161871, 161987 [MEP] Update placeholders before updating href (adobecom#3142) * MWPW-161590 [MEP] Update placeholders before updating href * unit test updates * MWPW-161871 [MEP] useblockcode action fails if page is on milo site (adobecom#3161) * stash * ready to publish * interact call coming back but timing out * working on a fast connection * clean up ifs * working with promise * use camel case on let variable * unit tests * update unit tests * MWPW-149504 [MILO][MEP] Move entitlements object to the same JSON file used by the library (adobecom#3047) * create mepxlg branch * update library * require full hostname match * update reference for unit test * switch to use config instead of domain list and stub response in unit test * update fetch to 2 * updating another fetch to 2 * restore normalizePath to use preview domains on preview links * preload segment list json * use getFederatedUrl instead * import at top so we don't have to make normalizePath async * MWPW-148129 [MILO][MEP][GNAV] Able to use select by url feature with federated link (adobecom#3064) * add federated link function to registerInBlockActions * add to unit test * add unit test coverage * initial commit * Fix useblock code issue --------- Co-authored-by: vgoodric <vgoodric@adobe.com> Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com> * MWPW-161606 [MEP] Modify within modals with MEP (adobecom#3171) initial commit publish Co-authored-by: viloria <viloria@adobe.com> * add conditionals --------- Co-authored-by: Mark Perry <124626043+markpadbe@users.noreply.github.com> Co-authored-by: viloria <viloria@adobe.com> * [MWPW-161782] - Added logo changes for older GNAV experience (adobecom#3162) * Added logo changes for older GNAV experience * Lint Fix * Logo alighned to left for older GNAV * [MWPW-161563] CDT Overlap in Global and APAC Promotion file (adobecom#3168) * Use schedule as source for CDT * seperate start and end * Added unit tests * [MWPW-157451] [Accessibility] Merch cards | Focus is lost on comparing plans section (adobecom#3103) * remive tabindex=0 as the attributes from merchcard div * fresh commit with same changes as commit 1 after resolving the merge conflict * Redesign mini-compare chart. Added checkmarks for icons. Backward compatibility using "checkmark" class. Horizontal line added. * Revert "Redesign mini-compare chart. Added checkmarks for icons. Backward compatibility using "checkmark" class. Horizontal line added." This reverts commit 610807a. * run build in mas * run build --------- Co-authored-by: Rohit Sahu <rosahu@adobe.com> * MWPW-159381: Replaces CSS 'body *' with '.consonant-Wrapper *' (adobecom#3152) * Revert "MWPW-156749: Fix video CLS " (adobecom#2899) (adobecom#2900) Revert "MWPW-156749: Fix video CLS (adobecom#2849)" This reverts commit d4134c8. * [MWPW-159903] Fix quiz video marquees (adobecom#3009) (adobecom#3013) fix quiz marquees * [MWPW-159328] handle a case where there are not placeholders availabl… (adobecom#3014) [MWPW-159328] handle a case where there are not placeholders available (adobecom#2998) * [MWPW-159328] handle a case where there are not palceholders availble * fixed typos --------- Co-authored-by: Denys Fedotov <denlight@gmail.com> Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local> * MWPW-146211 [MILO][MEP] Option to select all elements (adobecom#2976) (adobecom#3023) * stash * stash * stash * working well * set updated command list for inline * remove querySelector function * unit test and custom block fix * updates for in-block * merch-card-collection unit test fixed * unit test updates * more unit test repair * linting errors * more linting * Fix Invalid selector test * add coverage * force git checks to refire * remove comment * pass rootEl to getSelectedElements for use when needed (gnav) * skip if clause in codecov --------- Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com> Co-authored-by: markpadbe <markp@adobe.com> * MWPW-158455: Promobar overlays with localnav elements in devices (adobecom#2991) * Revert "MWPW-156749: Fix video CLS " (adobecom#2899) (adobecom#2900) Revert "MWPW-156749: Fix video CLS (adobecom#2849)" This reverts commit d4134c8. * Changing z-index of promobar and popup * Changing z-index of promobar and popup * Reverting z-index to 4 for promobar --------- Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com> Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com> Co-authored-by: Blaine Gunn <Blainegunn@gmail.com> Co-authored-by: Akansha Arora <> * Replaces CSS 'body *' with '.consonant-Wrapper *' --------- Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com> Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com> Co-authored-by: Blaine Gunn <Blainegunn@gmail.com> Co-authored-by: Denys Fedotov <denlight@gmail.com> Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local> Co-authored-by: Rares Munteanu <overmyheadandbody@gmail.com> Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com> Co-authored-by: markpadbe <markp@adobe.com> Co-authored-by: Akansha Arora <akanshaarora090@gmail.com> * [MWPW-160948] Remove blockquote default spacing (adobecom#3159) * [MWPW-161098] Environment-aware links support for CaaS (adobecom#3165) * relative urls caas support * added unit tests * [MWPW-162059] - Fix for vertical alignment of logo on AH (adobecom#3180) Fix for verticle alignment of logo * Global links underlined by default (adobecom#3166) * a links underline, a.quite added * added styles to omit header/footer * added default link styles for headlines and lockup area * minor :hover style alignment on cards-horiz, marquee-anchors, brick * removed dubs underline on links * removed styles overrides to support double in horiz-card, marquee-anch, brick * [MWPW-145672] Remove Gnav global link styles * [MWPW-161624] Link farm double underline on focus * Ensure sub text in anchors is underlined correctly * Ensure sub text in anchors is underlined correctly --------- Co-authored-by: thi64146 <thi64146@adobe.com> * Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (adobecom#3183) Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (adobecom#2986)" This reverts commit f379815. Co-authored-by: Rares Munteanu <overmyheadandbody@gmail.com> * [MWPW-145672] Change link color only on hover (adobecom#3186) --------- Co-authored-by: Dev Ashish Saradhana <41122199+Deva309@users.noreply.github.com> Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com> Co-authored-by: Mark Perry <124626043+markpadbe@users.noreply.github.com> Co-authored-by: viloria <viloria@adobe.com> Co-authored-by: Rahul Gupta <rahulgupta999@gmail.com> Co-authored-by: rohitsahu <sahurohit9@gmail.com> Co-authored-by: Rohit Sahu <rosahu@adobe.com> Co-authored-by: cmiqueo <64917520+cmiqueo@users.noreply.github.com> Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com> Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com> Co-authored-by: Blaine Gunn <Blainegunn@gmail.com> Co-authored-by: Denys Fedotov <denlight@gmail.com> Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local> Co-authored-by: Rares Munteanu <overmyheadandbody@gmail.com> Co-authored-by: markpadbe <markp@adobe.com> Co-authored-by: Akansha Arora <akanshaarora090@gmail.com> Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com> Co-authored-by: thi64146 <thi64146@adobe.com> Co-authored-by: Narcis Radu <github@narcisradu.ro>
|
sheridansunier
changed the title
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## stage #3224 +/- ##
=======================================
Coverage 96.37% 96.38%
=======================================
Files 245 245
Lines 56716 56715 -1
=======================================
+ Hits 54661 54662 +1
+ Misses 2055 2053 -2 ☔ View full report in Codecov by Sentry. |
sanrai
approved these changes
Nov 20, 2024
gmirijan
added
caas-configurator
gmirijan
approved these changes
Nov 26, 2024
|
Reminder to set the |
chrischrischris
approved these changes
Nov 27, 2024
|
This PR has not been updated recently and will be closed in 7 days if no action is taken. Please ensure all checks are passing, https://github.com/orgs/adobecom/discussions/997 provides instructions. If the PR is ready to be merged, please mark it with the "Ready for Stage" label. |
|
Closing this PR due to inactivity. |
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.
As a PM, I would like the ability to pick and choose various event states that are included in my card collection, so we can have the flexibility to include and exclude event states. Currently right now, an author can only select an event state of either All or Live or Upcoming or On-demand.
This change includes the collection authoring changes required to enable this behavior.
REQUIRES: adobecom/caas#197
Resolves: MWPW-155425
Test URLs: