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

Merging inline-block to milo branch for further updates #2170

Merged
merged 6 commits into from
Apr 19, 2024

Conversation

vgoodric
Copy link
Contributor

Merging inline-block to milo branch for further updates

@vgoodric vgoodric requested review from a team as code owners April 19, 2024 23:36
@vgoodric vgoodric merged commit 1e038be into adobecom:gnavinternal Apr 19, 2024
13 of 14 checks passed
mokimo added a commit that referenced this pull request May 8, 2024
… the gnav file (#2252)

* Merging inline-block to milo branch for further updates (#2170)

* MWPW-135821 introduce custom action

- and inaugurating first custom action for replacing cards in collection

* MWPW-135821 use prefix in selector

* MWPW-135821 adding preview data

---------

Co-authored-by: Nicolas Peltier <npeltier@adobe.com>
Co-authored-by: Nicolas Peltier <1032754+npeltier@users.noreply.github.com>

* initial commit with TODO

* registerCustomAction updated

* fragment, highlight fragment and merch-card update

* chain fragment support added

* inBlock fragments with URL and path as keys

* css selectors starting to work

* Add forceInline to handleCommands

* stash

* unit tests mostly done

* fix mocking issue

* add martech-metadata to unit test mock and check for replace

* change name of registerCustomAction to registerInBlockActions

* indent issue

* remove mep useBlockCode Example

* merch-card-collection-fix

* update merch-card-collection unit test

* ignore lines that cannot be reached by unit tests

* rename CUSTOM_SELECTOR_PREFIX

* Update libs/features/personalization/personalization.js

Co-authored-by: Chris Peyer <chrischrischris@users.noreply.github.com>

* Update libs/features/personalization/personalization.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update libs/features/personalization/personalization.js

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* call addHash once, not twice

* Update libs/features/personalization/personalization.js

Co-authored-by: Chris Peyer <chrischrischris@users.noreply.github.com>

---------

Co-authored-by: Nicolas Peltier <npeltier@adobe.com>
Co-authored-by: Nicolas Peltier <1032754+npeltier@users.noreply.github.com>
Co-authored-by: markpadbe <markp@adobe.com>
Co-authored-by: Chris Peyer <chrischrischris@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants