ondrej
Février 4, 2026, 7:47
1
Dans la page d’administration des utilisateurs, si vous faites défiler jusqu’à la section « Groupes » et essayez d’ajouter l’utilisateur à un groupe personnalisé, le bouton apparaîtra comme vous pouvez le voir dans l’image ci-dessous.
Cependant, lorsque vous appuyez dessus, il ne disparaît pas, donnant l’impression que l’action n’a pas été effectuée. Mais si vous actualisez la page, l’utilisateur a bien été ajouté au groupe, mais l’interface utilisateur ne le reflète pas.
2 « J'aime »
Une autre page qui serait mieux adaptée à un véritable refactoring vers Glimmer
En attendant, voici la correction
main ← fix/admin-user-custom-groups-save
opened 11:13PM - 07 Feb 26 UTC
The approve/reject buttons for custom group changes on the admin user page would… remain visible after saving, even though the changes were persisted successfully. Removing a group also failed to update the UI.
Three issues in the reactivity chain caused this:
1. The `@filter` computed macros (classic Ember) on the AdminUser model did not react to `@trackedArray` mutations. Replaced `customGroups` and `automaticGroups` with native getters so they participate in Glimmer autotracking.
2. The controller's `groupAdded()` and `groupRemoved()` wrappers did not return their promise chains, so `await` in `saveCustomGroups()` resolved immediately before the API calls completed and before the groups array was updated.
3. The controller's `customGroupIds`, `customGroupsDirty`, and `automaticGroups` used `@discourseComputed` which could not track changes from native getters on the model. Converted these to native getters and made `customGroupIdsBuffer` `@tracked`.
Additionally, `saveCustomGroups()` now awaits each operation sequentially and calls `resetCustomGroups()` on completion to sync the buffer with the updated group list, clearing the dirty state.
Note: only the properties involved in the groups reactivity chain were converted to modern Glimmer patterns. The rest of the controller and model still use classic Ember (`@discourseComputed`, `this.set`, etc.) and can be modernized separately.
Ref - https://meta.discourse.org/t/395210
1 « J'aime »
zogstrip
A fermé ce sujet ()
Février 14, 2026, 7:00
5
Ce sujet a été automatiquement fermé après 2 jours. Les nouvelles réponses ne sont plus autorisées.