MWPW-161130 - [Table] Handle dynamic pricing in addon labels

[Sartxi]

When adding dynamic prices in the table blocks new addon feature the dynamic price element is rendered in the flex grid instead of inline with the text content. Fixing this issue by separating out the icon element and grouping all other authored content in addon labels.

Resolves: MWPW-161130

Test URLs:

• Before: https://main--milo--adobecom.hlx.page/drafts/sarchibeque/table/add-on?martech=off
• After: https://sartxi-mwpw-161130-table--milo--adobecom.hlx.page/drafts/sarchibeque/table/add-on?martech=off

[aem-code-sync]

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

• e1cee36 ✅ (latest)
• 7d20f24 ✅
• c1e148b ✅
• 7e9b4cc ✅
• ee6c579 ✅

[aem-code-sync]

	Page	Scores	Audits	Google
📱	/drafts/sarchibeque/table/add-on?martech=off
🖥️	/drafts/sarchibeque/table/add-on?martech=off

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.35%. Comparing base (8f0adc1) to head (e1cee36).
Report is 16 commits behind head on stage.

Additional details and impacted files

@@            Coverage Diff             @@
##            stage    #3100      +/-   ##
==========================================
+ Coverage   96.34%   96.35%   +0.01%     
==========================================
  Files         245      245              
  Lines       56316    56375      +59     
==========================================
+ Hits        54256    54321      +65     
+ Misses       2060     2054       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

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.

[robert-bogos]

Hi @Sartxi ! Are the failing unit tests related to your changes?

[Sartxi]

Hi @Sartxi ! Are the failing unit tests related to your changes?

I was able to get it passing by rebasing with Stage!

[robert-bogos]

Looks good 👍

[github-actions]

Reminder to set the Ready for Stage label - to queue this to get merged to stage & production.

[NadiiaSokolova]

Verified. Ready for Stage.
Testing details https://jira.corp.adobe.com/browse/MWPW-161130