Ça a l’air super ! 
Peux-tu confirmer que les paramètres fonctionnent également ?
5 « J'aime »
Oui, ils s’affichent dans l’interface utilisateur si la requête contient des paramètres.
5 « J'aime »
Je me suis trompé ; les paramètres ne fonctionnent pas actuellement. Je vais examiner cela aujourd’hui.
L’adaptateur pour les requêtes est un adaptateur administrateur, de sorte que les utilisateurs non administrateurs ne peuvent pas accéder au magasin de requêtes. J’ai ajouté des points de terminaison que les non-administrateurs peuvent utiliser pour obtenir le JSON de la requête, mais il semble que ce JSON ne soit pas converti en modèles de requête (bien sûr, comment pourraient-ils l’être), et donc la logique des paramètres n’est pas exécutée. Je suppose qu’il me faut un adaptateur non administrateur pour les requêtes ?
2 « J'aime »
Je vois un problème dans la capture d’écran n°3 : vous devriez peut-être rétablir le bouton « Enregistrer sans exécuter », maintenant qu’il y a des données non liées aux requêtes à sauvegarder.
Ah, je n’avais pas remarqué que vous aviez également importé le bouton de confirmation vert.
3 « J'aime »
Le bouton vert de confirmation enregistre sans exécuter, et je pensais que le bouton « Enregistrer les modifications et exécuter » ferait exactement cela.
Voulez-vous dire que le comportement de « Enregistrer les modifications et exécuter » devrait changer et ne pas exécuter la requête ?
3 « J'aime »
Les membres du groupe peuvent désormais exécuter des requêtes avec des paramètres !
Je n’arrive pas à lancer les spécifications (avec bundle exec rspec), et donc je n’ai pas écrit de tests pour les nouvelles actions que j’ai créées. Quelqu’un peut-il m’indiquer ce qui me manque ? Le message d’erreur est uninitialized constant DataExplorer.
4 « J'aime »
Essayez LOAD_PLUGINS=1 RAILS_ENV=test bundle exec rspec
5 « J'aime »
Oui, ça marche. Merci !
5 « J'aime »
Notez que j’utilise généralement bin/rake autospec, ce qui le fait automatiquement. Tout ce que vous avez à faire est d’enregistrer plugin.rb ou quelque chose de similaire.
5 « J'aime »
@sam,
Que penses-tu de la case à cocher « Inclure le plan de requête ? » pour les rapports de groupe ?
Il semble que les membres du groupe n’aient pas besoin d’y avoir accès et qu’elle devrait être supprimée. Cette hypothèse est-elle correcte ?
5 « J'aime »
Je suis d’accord, c’est une information dont ils n’ont pas vraiment besoin.
6 « J'aime »
Cela a été supprimé et la PR est prête pour examen.
2 tests échouent, mais ils échouent également sur la branche master. Je pourrais examiner la correction de ces échecs dans cette PR si cela est apprécié 
5 « J'aime »
Tout à fait le bienvenu ! Nous l’examinerons sous peu, merci pour ce travail formidable.
3 « J'aime »
J’essaie d’utiliser cette fonctionnalité, mais il semble que les groupes que j’ajoute au formulaire ne soient pas conservés, c’est-à-dire qu’il n’y a aucun moyen d’enregistrer le formulaire.
Voici une capture d’écran de la requête que je souhaite partager avec les modérateurs.
J’ajoute un groupe comme ceci :
Et lorsque je rafraîchis la page, le groupe a disparu :
Modifier et « enregistrer » la requête avant de rafraîchir n’a aucun effet non plus. J’ai également essayé de partager d’autres requêtes, y compris certaines des valeurs par défaut du système, sans succès.
1 « J'aime »
Une petite mise à jour : j’ai un autre site où cela fonctionne, donc le problème semble spécifique au site.
Quoi qu’il en soit, j’ai trouvé une solution de contournement si l’interface utilisateur est également défectueuse pour vous : vous pouvez ajouter un groupe à la requête via l’API en envoyant une requête POST vers https://example.com/admin/plugins/explorer/queries/<query_id>.
Configurez vos en-têtes et utilisez une clé API pour un utilisateur administrateur :
Api-UsernameApi-KeyContent-Type– application/json
Configurez le corps de votre requête :
query[group_ids][]– chaîne de caractères séparée par des virgules contenant les identifiants des groupes autorisés à accéder à cette requête.
1 « J'aime »
Je regarderai aujourd’hui ou lundi.
6 « J'aime »
Pas urgent,
Merci @j.jaffeux !
1 « J'aime »
Cela devrait être corrigé par :
Merci pour le signalement.
7 « J'aime »
Ce sujet a été automatiquement fermé 2 jours après la dernière réponse. De nouvelles réponses ne sont plus autorisées.