[beta5] consent manager: fix sub services ID

Created on 11 October 2023, about 1 year ago

When we did 🐛 [beta2] ⚠️ Fix modal structure in consent_banner pattern Fixed we introduce a new structure for the services list, closer to the standard Drupal menu structure.

However, we have updated the Twig template only for the first level of this menu.

So, today, the sub services ID are wrong. Instead of the expected ID:

<legend id="google-analitycs-legend" class="fr-consent-service__title">Google Analytics</legend>

We have integer increment:

<legend id="google-analitycs-0" class="fr-consent-service__title">Google Analytics</legend>

Fix the second level loop to use sub_service.attributes.id instead of the array key.

🐛 Bug report

Status

Fixed

Version

1.0

Component

Code

Created by

🇫🇷France pdureau Paris

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @pdureau
Comment about 1 year ago →
System Message

pdureau → committed d7720aa0 on 1.0.x
Issue #3393343 by pdureau: [beta5] consent manager: fix sub services ID
Issue was unassigned.
Status changed to Fixed about 1 year ago3:06pm 11 October 2023
Comment about 1 year ago →
🇫🇷France pdureau Paris
Fixe din commit d7720aa
Comment about 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024