Skip to content

N°9172 Modify ext mgmt labels#921

Merged
Timmy38 merged 3 commits into
feature/uninstallationfrom
issue/9172_change_labels
May 28, 2026
Merged

N°9172 Modify ext mgmt labels#921
Timmy38 merged 3 commits into
feature/uninstallationfrom
issue/9172_change_labels

Conversation

@Timmy38
Copy link
Copy Markdown
Contributor

@Timmy38 Timmy38 commented May 28, 2026

No description provided.

@Timmy38 Timmy38 self-assigned this May 28, 2026
@Timmy38 Timmy38 added the internal Work made by Combodo label May 28, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 28, 2026

Greptile Summary

This PR updates the feature removal flow into an extension management flow. It changes:

  • User-facing English and French labels for extension management.
  • Main and analysis Twig panels to show revised titles and helper text.
  • Setup data-audit wording and cleanup action labels.
  • The advanced force-uninstall setup option to show its warning when already enabled.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Reviews (2): Last reviewed commit: "N°9172 Modify ext mgmt labels" | Re-trigger Greptile

Comment thread setup/wizardsteps/AbstractWizStepMiscParams.php Outdated
@Timmy38 Timmy38 closed this May 28, 2026
@Timmy38 Timmy38 reopened this May 28, 2026
@Timmy38 Timmy38 merged commit 683b075 into feature/uninstallation May 28, 2026
1 check passed
@Timmy38 Timmy38 deleted the issue/9172_change_labels branch May 28, 2026 09:19
'DataFeatureRemoval:Helper:Desc1' => 'It will prepare the setup step that proceeds to feature enabling or disabling.',
'DataFeatureRemoval:Helper:Desc2' => 'Analyze if there are any data or dependency preventing you from enabling/disabling a feature.',
'DataFeatureRemoval:Main:Title' => 'Extension management',
'DataFeatureRemoval:Main:SubTitle' => 'Toggle extensions installed on your iTop',
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.

Shouldn't this use a variable instead of "iTop"?

'DataFeatureRemoval:Main:Title' => 'Extension management',
'DataFeatureRemoval:Main:SubTitle' => 'Toggle extensions installed on your iTop',
'DataFeatureRemoval:Failure:Title' => 'Extensions dry removal errors',
'DataFeatureRemoval:Helper:Title' => 'Analyze if there are any data or dependency preventing you from adding/removing an extension.',
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.

I'd suggest "dependencies"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Work made by Combodo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants