Skip to content
Snippets Groups Projects
This project is mirrored from https://github.com/metabase/metabase. Pull mirroring updated .
  1. Nov 14, 2024
  2. Nov 13, 2024
  3. Nov 12, 2024
  4. Nov 11, 2024
  5. Nov 10, 2024
  6. Nov 08, 2024
  7. Nov 07, 2024
  8. Nov 06, 2024
    • adam-james's avatar
      Filter out Empty Rows from Pivot Exports (#49512) · 63151f4e
      adam-james authored
      * Pivot Measures Order Used in Pivot Exports
      
      Fixes #48442
      
      A pivot table can have any number of measures, and the user can order these by dragging in the UI. Before this PR,
      that order was ignored and measures would alway be in index order, which is confusing for any user who needs the
      measures to be displayed in a particular order, especially if they've re-ordered them in the pivot viz settings UI.
      
      A test has been added to check that measure order is used.
      
      A few minor changes to the pivot qp and post-processor
       - measure indices are looked up in the pivot qp and added in viz-settings order
       - the pivot measures are only added if the qp has not already added them.
       - pivot-opts Malli spec has been made in the namespace, adjusted to allow `nil` as a valid pivot-opts output, and
       used in relevant functions
      
      * address review points.
      
      * add a rows order test
      
      * Filter out Empty Rows from Pivot Exports
      
      Fixes #49353
      
      The linked issue is not actually related to pivot export size but is instead related to the 'Min of Created At: Month'
      aggregation; the default aggregation function was `+`, so it broke when the date string was encountered.
      
      That was fixed.
      
      As I was trynig to keep the export small if possible, I noticed that in some cases empty rows are appended, so I added
      the filter so that if a pivot row's values are completely empty, it doesn't add it.
      
      Finally, this PR also adds the 'sub section totals' which I noticed were missing from the exports. This comes up when
      you have 3+ pivot-rows configured, so you can see the subtotals for the first pivot row and the subtotals nested
      within those sections for the second pivot row, and so on.
      
      * add test for non-numeric values
      
      * Make sure the new name refs match on aggregations not just breakouts
      
      * cljfmt
      63151f4e
    • Noah Moss's avatar
    • Alexander Polyankin's avatar
      Fix :pivot-measures with column names (#49575) · 240634d3
      Alexander Polyankin authored
      
      * Fix :pivot-measures with column names
      
      * Fix the test
      
      * Update src/metabase/query_processor/pivot.clj
      
      Co-authored-by: default avatarBraden Shepherdson <braden@metabase.com>
      
      * Add a test
      
      * Add a test
      
      ---------
      
      Co-authored-by: default avatarBraden Shepherdson <braden@metabase.com>
      240634d3
    • Braden Shepherdson's avatar
      [testing] Add filters to random query generators (#49448) · 46191072
      Braden Shepherdson authored
      Adds filters to generative testing for queries.
      
      There's a lot of new code here, but don't panic, it's quite straightforward:
      generating filter values, in the right shapes for various filter expressions.
      46191072
    • Ngoc Khuat's avatar
      [Notification] Pulse clean up 2 (#49519) · 3e33ab4c
      Ngoc Khuat authored
      3e33ab4c
    • Ngoc Khuat's avatar
      [Notification] Pulse clean up (#49419) · a0598e46
      Ngoc Khuat authored
      a0598e46
  9. Nov 05, 2024
Loading