Skip to content

Commit 5c246ae

Browse files
ccaillycedric-anne
authored andcommitted
fix(helpdesk): Rename 'link' to 'url' in external page tile configuration
1 parent 42a714b commit 5c246ae

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

templates/pages/admin/external_page_tile_config_fields.html.twig

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@
6060
) }}
6161

6262
{{ fields.urlField(
63-
'link',
63+
'url',
6464
tile.getTileUrl(),
6565
__('Target url'),
6666
{

tests/cypress/e2e/self-service/home_config.cy.js

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -274,6 +274,11 @@ for (const test of tests) {
274274
"My external tile title",
275275
]);
276276
cy.findByText("My description").should('be.visible');
277+
validateTileFields(
278+
"My external tile title",
279+
"My description",
280+
"support.teclib.com"
281+
);
277282
});
278283

279284
it(`can add a "Form" tile (${test.label})`, () => {
@@ -475,6 +480,13 @@ function validateTilesOrder(tiles) {
475480
});
476481
}
477482

483+
function validateTileFields(title, description, target) {
484+
cy.findByRole("region", {'name': title}).click();
485+
cy.findByRole("heading", {'name': title}).should('be.visible');
486+
cy.findByLabelText('Description').awaitTinyMCE().should('contain', description);
487+
cy.findByLabelText('Target url').should('have.value', target);
488+
}
489+
478490
function moveTileAfterTile(subject, destination) {
479491
cy.findByRole("region", {'name': subject}).startToDrag();
480492
cy.findByRole("region", {'name': destination}).dropDraggedItemAfter();

0 commit comments

Comments
 (0)