-
Notifications
You must be signed in to change notification settings - Fork 176
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-155566] [MEP] Allow relevant actions in MEP to use text and HTML instead of just fragments #2683
Conversation
Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
Commits
|
|
This pull request is not passing all required checks. Please see this discussion for information on how to get all checks passing. Inconsistent checks can be manually retried. If a test absolutely can not pass for a good reason, please add a comment with an explanation to the PR. |
Resolves:
MWPW-155566
Also resolves:
MWPW-148285 [MILO][MEP] Add simplified selector to select the paragraph around a block's links - one of the best use cases for the update action, so was added to this PR
MWPW-148284 [MILO][MEP] Allow selectors to have a mix of CSS and simplified selectors - update for 148285 also ended up solving this
URL for testing:
Psi-check: https://mwpw-148284--milo--adobecom.hlx.page/?martech=off
Recent updates to this PR:
updated the getSelectedElement function to have a different approach:
3.updated the getSelectedElement function to have a different approach:
benefits to this approach: