chore(TwoWindingsTransformerCreation): use form from commons-UI

[dbraquart]

PR Summary

TwoWindingsTransformerCreation form is now supported in composite modifications dialog. Requires gridsuite/commons-ui#1294.

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 26b2bb37-0a07-46f4-b22d-7f268200b986

📝 Walkthrough

Walkthrough

The composite modification dialog now supports two-windings transformer creation. The modification dialog accepts an optional breakpoint width and passes it to CustomMuiDialog.

Changes

Two-windings transformer creation

Layer / File(s)	Summary
Configurable modification dialog width src/components/dialogs/network-modification/simple-modification/ModificationDialog.tsx, src/components/dialogs/network-modification/composite-modification/composite-modification-dialog.tsx	ModificationDialog accepts an optional dialogWidth breakpoint and passes it to CustomMuiDialog. The composite dialog configuration supports this property.
Composite dialog creation wiring src/components/dialogs/network-modification/composite-modification/composite-modification-dialog.tsx	The dialog imports and registers the two-windings transformer creation form, schema, and DTO conversion functions. It adds the creation identifiers, settings, and XL dialog width.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly identifies the main change: using the commons-UI form for TwoWindingsTransformerCreation.
Description check	✅ Passed	The description accurately states that TwoWindingsTransformerCreation is supported in the composite modifications dialog.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In
`@src/components/dialogs/network-modification/composite-modification/composite-modification-dialog.tsx`:
- Around line 232-244: Add the missing TwoWindingsTransformerCreationError and
CreateTwoWindingsTransformer localization keys to the app’s translation
catalogs, ensuring the existing errorHeaderId and titleId references in the
composite modification configuration resolve to translated messages.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: cdd3cd9b-8efa-4594-8cb7-8178b5259b56

📥 Commits

Reviewing files that changed from the base of the PR and between 8cce03c and d066011.

📒 Files selected for processing (1)

• src/components/dialogs/network-modification/composite-modification/composite-modification-dialog.tsx

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In
`@src/components/dialogs/network-modification/simple-modification/ModificationDialog.tsx`:
- Around line 107-109: Update ModificationDialog’s width props and
.MuiDialog-paper minWidth styling to apply only when dialogWidth is provided;
preserve the existing default dialog sizing when it is omitted, including
transformer-creation dialogs. Then verify desktop layouts for legacy dialogs
that still use the base dialog style.

🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b72b409a-5d7c-4961-9ae3-18e543c30d98

📥 Commits

Reviewing files that changed from the base of the PR and between 79a3720 and 73238bc.

📒 Files selected for processing (2)

• src/components/dialogs/network-modification/composite-modification/composite-modification-dialog.tsx
• src/components/dialogs/network-modification/simple-modification/ModificationDialog.tsx

🚧 Files skipped from review as they are similar to previous changes (1)

• src/components/dialogs/network-modification/composite-modification/composite-modification-dialog.tsx

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud