cube-js/cube

`shouldFetchDryRun` should consider changes made to filters

Open

#3 552 ouverte le 18 oct. 2021

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Rust (1 965 forks)batch import
client:reacthelp wanted

Métriques du dépôt

Stars
 (19 563 stars)
Métriques de merge PR
 (Merge moyen 5j 16h) (138 PRs mergées en 30 j)

Description

Describe the bug When I modify filters using updateFilters methods within QueryBuilder, a dry run does not occur as expected. Adding filters to the existing shouldFetchDryRun array (['measures', 'dimensions', 'timeDimensions']) fixes the issue.

To Reproduce Steps to reproduce the behavior:

  1. Call updateFilters.add QueryBuilder (@cubejs-client/react) to modify filters
  2. A dry run does not occur

Expected behavior I expect a filter change to prompt a dry run.

Screenshots https://github.com/cube-js/cube.js/blob/master/packages/cubejs-client-react/src/QueryBuilder.jsx#L485-L488

Version: 0.28.43

Additional context Add any other context about the problem here.

Guide contributeur

Direction de recherche
Localisez le tableau shouldFetchDryRun dans packages/cubejs client react/src/QueryBuilder.jsx autour de la ligne 485 et ajoutez 'filters' au tableau.
Stack technique
javascripttypescriptreact
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90