chore(forms): Rename 'Add strategy' button to 'Combine with another o…

…ption' for clarity
glpi-project · Jan 6, 2025 · 6c69bd8 · 6c69bd8
1 parent 740191e
commit 6c69bd8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/templates/pages/admin/form/form_destination_commonitil_config.html.twig b/templates/pages/admin/form/form_destination_commonitil_config.html.twig
@@ -167,11 +167,11 @@
                         <button
                             type="button"
                             class="btn btn-outline"
-                            aria-label="{{ __('Add strategy') }}"
+                            aria-label="{{ __('Combine with another option') }}"
                             data-glpi-itildestination-add-field-config
                         >
                             <i class="ti ti-plus me-2"></i>
-                            {{ __('Add another strategy') }}
+                            {{ __('Combine with another option') }}
                         </button>
                     {% endif %}
                 {% else %}

diff --git a/tests/cypress/e2e/form/destination_config_fields/associated_items.cy.js b/tests/cypress/e2e/form/destination_config_fields/associated_items.cy.js
@@ -280,7 +280,7 @@ describe('Associated items configuration', () => {
         });
 
         // Add strategy
-        cy.get('@config').findByRole('button', {'name': 'Add strategy'}).click();
+        cy.get('@config').findByRole('button', {'name': 'Combine with another option'}).click();
         cy.get('@config').find('[data-glpi-itildestination-field-config]').eq(1).getDropdownByLabelText('Select strategy...').as('specific_associated_items_strategy_dropdown');
         cy.get('@specific_associated_items_strategy_dropdown').selectDropdownValue('Answer from specific questions');
 

diff --git a/tests/cypress/e2e/form/form_destination.cy.js b/tests/cypress/e2e/form/form_destination.cy.js
@@ -147,7 +147,7 @@ describe('Form destination', () => {
 
     it('can define multiple strategies for the same field', () => {
         cy.findByRole('region', {name: 'Requesters configuration'}).as('requesters_config');
-        cy.get('@requesters_config').findByRole('button', {name: 'Add strategy'}).should('exist').as('add_strategy_button');
+        cy.get('@requesters_config').findByRole('button', {name: 'Combine with another option'}).should('exist').as('add_strategy_button');
         cy.getDropdownByLabelText('Requesters').as('first_strategy_dropdown');
 
         // Define first strategy
@@ -178,7 +178,7 @@ describe('Form destination', () => {
         });
 
         // Add a fourth strategy
-        cy.get('@requesters_config').findByRole('button', {name: 'Add strategy'}).click();
+        cy.get('@requesters_config').findByRole('button', {name: 'Combine with another option'}).click();
         cy.findByRole('combobox', {name: '-----'}).as('fourth_strategy_dropdown');
         cy.get('@fourth_strategy_dropdown').selectDropdownValue('User who filled the form');