Skip to content

Fix emails plan advisor ranges pricing in NewPlanSelection#2887

Merged
jhoffman-ms merged 1 commit into
masterfrom
no-task-fix-issue-emails-plan-new-plan-selection
Jun 2, 2026
Merged

Fix emails plan advisor ranges pricing in NewPlanSelection#2887
jhoffman-ms merged 1 commit into
masterfrom
no-task-fix-issue-emails-plan-new-plan-selection

Conversation

@jhoffman-ms
Copy link
Copy Markdown
Contributor

Related

  • Issue/task: not provided

Source of truth

  • SPEC: docs/specs/SPEC_20260518_plan-envios-picture-13.md
  • This PR also updates that SPEC to explicitly cover the A medida* / Tailored* main-price state for the Emails plan advisor scenarios.

Relevant assets

  • docs/assets/picture_14.png
  • docs/assets/picture_15.png
  • Reference image shared in the task for the expected price/CTA block

What changed

  • Updated NewPlanSelection/EmailsPlan so that selecting Menos de 100.000 or Más de 10.000.000 switches the main price block to the commercial state:
    • price label stays the same
    • price value becomes A medida* / Tailored*
    • main CTA stays as Contactar a Asesor / Contact an Advisor
  • Hid the extra-email-price line in those commercial scenarios so the card matches the tailored-price layout.
  • Kept the sticky summary behavior unchanged in those scenarios, preserving the standard plan summary with commercial CTA as defined by the existing flow.
  • Added test coverage for both special Emails-plan ranges to assert the tailored-price state and hidden extra-email-price copy.
  • Updated the spec so the main-card tailored-price behavior is explicitly documented for both downgrade (< 100.000) and high-volume (> 10.000.000) states.

How it was tested

  • yarn test:related -- src/components/BuyProcess/NewPlanSelection/index.test.js src/components/BuyProcess/NewPlanSelection/EmailsPlan/index.js
  • yarn prettier-check

Open questions / follow-up

  • None for this change. The spec now matches the implemented UI behavior.

@jhoffman-ms jhoffman-ms force-pushed the no-task-fix-issue-emails-plan-new-plan-selection branch from 9421497 to 9049f12 Compare June 2, 2026 13:39
@jhoffman-ms jhoffman-ms merged commit f8dba12 into master Jun 2, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant