-
- Downloads
Add `parameters` to report_card (#22976)
* add parameters to report_card and card APIs * make sure migration can work with existing cards and add migration tests * remove debug code * When parameters is empty, return template tag parameters * update schema message * update migraiton test name * minor changes in test * fix failing tests * Always use card.parameters in PublicQuestion * Add defaults and readd isEmpty check * Default to undefined/use template tags in PublicQuestion... * parameters should be in the writableProperties * Native query card: construct parameters from template tags * Separate the generation of parameter id * Add parameter_mappings to report_card (#23003) * add parameter_mappings to report_card * fix from Noah's comments * fix from Noah's comments * Update a parameter from an updated template tag * Correct the parameters construction * Also add `parameter_mappings` to writableProperties * CI test: bust the npm module cache * Revert "CI test: bust the npm module cache" This reverts commit 5a327b616f0220f43a90f7f871e0bd877ffa6f47. Co-authored-by:Dalton Johnson <daltojohnso@users.noreply.github.com> Co-authored-by:
Ariya Hidayat <ariya@metabase.com>
Showing
- enterprise/backend/test/metabase_enterprise/serialization/cmd_test.clj 2 additions, 0 deletions...ckend/test/metabase_enterprise/serialization/cmd_test.clj
- frontend/src/metabase/entities/questions.js 2 additions, 0 deletionsfrontend/src/metabase/entities/questions.js
- frontend/src/metabase/parameters/utils/cards.ts 24 additions, 17 deletionsfrontend/src/metabase/parameters/utils/cards.ts
- frontend/src/metabase/parameters/utils/dashboards.ts 2 additions, 1 deletionfrontend/src/metabase/parameters/utils/dashboards.ts
- frontend/src/metabase/parameters/utils/parameter-id.js 4 additions, 0 deletionsfrontend/src/metabase/parameters/utils/parameter-id.js
- frontend/src/metabase/parameters/utils/parameter-id.unit.spec.js 7 additions, 0 deletions...d/src/metabase/parameters/utils/parameter-id.unit.spec.js
- frontend/src/metabase/public/containers/PublicQuestion.jsx 11 additions, 3 deletionsfrontend/src/metabase/public/containers/PublicQuestion.jsx
- frontend/src/metabase/query_builder/actions/native.js 28 additions, 2 deletionsfrontend/src/metabase/query_builder/actions/native.js
- resources/migrations/000_migrations.yaml 55 additions, 0 deletionsresources/migrations/000_migrations.yaml
- src/metabase/api/card.clj 12 additions, 7 deletionssrc/metabase/api/card.clj
- src/metabase/api/dashboard.clj 2 additions, 2 deletionssrc/metabase/api/dashboard.clj
- src/metabase/models/card.clj 18 additions, 7 deletionssrc/metabase/models/card.clj
- src/metabase/models/dashboard.clj 2 additions, 7 deletionssrc/metabase/models/dashboard.clj
- src/metabase/models/params.clj 14 additions, 0 deletionssrc/metabase/models/params.clj
- src/metabase/util/schema.clj 12 additions, 0 deletionssrc/metabase/util/schema.clj
- test/metabase/api/card_test.clj 64 additions, 1 deletiontest/metabase/api/card_test.clj
- test/metabase/api/dashboard_test.clj 2 additions, 1 deletiontest/metabase/api/dashboard_test.clj
- test/metabase/db/schema_migrations_test.clj 49 additions, 2 deletionstest/metabase/db/schema_migrations_test.clj
- test/metabase/events/revision_test.clj 2 additions, 0 deletionstest/metabase/events/revision_test.clj
- test/metabase/models/card_test.clj 69 additions, 0 deletionstest/metabase/models/card_test.clj
Loading
Please register or sign in to comment