Skip to content

Imp simulation prices rd88#238

Open
JoanaFigueira wants to merge 5 commits into
masterfrom
IMP_simulation_prices_rd88
Open

Imp simulation prices rd88#238
JoanaFigueira wants to merge 5 commits into
masterfrom
IMP_simulation_prices_rd88

Conversation

@JoanaFigueira

@JoanaFigueira JoanaFigueira commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Description

Add simulation section to review page

Changes

  • New simulation section in summary page
  • Fix Other Concepts section to match figma

Checklist

Justify any unchecked point:

  • Changed code is covered by tests.
  • Relevant changes are explained in the "Unreleased" section of the CHANGES.md file.
  • That section includes "Upgrade notes" with any config, dependency or deploy tweek needed on development and server setups.
  • Changes on the setup process (development, testing, production) have been updated in the proper documentation

Observations

Please, review

  • The simulation values are properly set for a home or company new contracts, as well as for indexed or periods type of tariff.

How to check the new features

Fill in new contract form and go to review page. You also need to connect to your local backend (at NEW_simulation_prices_rd88 branch ).

@vercel

vercel Bot commented Jun 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
webforms-ui Ready Ready Preview, Comment Jun 12, 2026 6:13am

Request Review

{t('SIMULATION_PRICES_TITLE')}
</Typography>
<Box
component="ul"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This text is not a list

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

upsi... copy paste... 🙈 Thanks!!

})
.then((response) => {
const tariffPrices = response?.data['current']
const estimatedMonthlykWh=response?.data['estimated_monthly_kwh']

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we set as 0, or ??, by default and sets only if backend returns something

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This screen could be like a "información precontractual", maybe is better ?? or XX to avoid lying with a simulation of 0€

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