Multiple fixes for v4.0.2 #1373
Conversation
… we can't use an external events manager, use new setModalAction function.
| const endUrl = button.getAttribute("data-bs-meeting-end-url"); | ||
| const modalHref = button.getAttribute("data-bs-meeting-info-url"); | ||
| const isWebinar = button.getAttribute("data-bs-meeting-webinar") == "True"; | ||
| const toggleChatUrl = button.getAttribute("data-bs-meeting-toggle-chat-url"); |
There was a problem hiding this comment.
pourquoi utiliser des "data-bs-" ... "bs" étant réservé aux éléments BootStrap, je préconise plutot un autre nom, comme "data-pod-..." par exemple.
There was a problem hiding this comment.
En fait, c'était déjà comme cela : tous les autres boutons utilisent déjà des attributs data-bs-*.
Cela aurait été bizarre de rajouter un nouveau bouton qui n'a pas le même nom de classe que les autres.
Et modifier tous les autres impliqueraient un paquet de modifications/tests complémentaires...
Badatos
left a comment
Badatos
left a comment
There was a problem hiding this comment.
Merci pour ces fixes !
Voici quelques remarques :
|
|
||
| let folder_id = button.dataset.folderid; | ||
| let filetype = button.dataset.filetype; | ||
| // Worjing variables |
There was a problem hiding this comment.
Oj, modifié :)
|
J'ai mis à jour la branche avec dev_v4 pour intégrer les FIX de la PR #1374 afin de voir si tout passe bien cette fois. |
|
Normalement, la CI devrait passer. On verra d'ici 15 minutes :) |
YES ! |
2 participants
Multiple fixes for v4.0.2:
Correct Podfile module:
Modify meeting module:
Remove |title (useful for exemple for 'IA') for a live event