Skip to content
Snippets Groups Projects
This project is mirrored from https://github.com/metabase/metabase. Pull mirroring updated .
  1. Oct 24, 2022
  2. Oct 22, 2022
  3. Oct 21, 2022
    • Nemanja Glumac's avatar
    • Maz Ameli's avatar
      Tweak copy and add educational link to the New -> Model screen (#26047) · 8793c847
      Maz Ameli authored
      * update new model copy
      
      * add educational button
      
      * remove EducationalButtonProps
      Unverified
      8793c847
    • Cam Saul's avatar
      Add `created_at` to Collection (#25871) · 7af4f706
      Cam Saul authored
      * Add created_at to Collection
      
      * Simplified migrations [ci skip]
      
      * Add SQL for H2 and MySQL; add tests
      
      * Sort namespaces
      
      * Test fixes :wrench:
      
      * Test fixes
      
      * Test fixes :wrench:
      
      * Make sure MySQL/MariaDB Liquibase generator sets the current timestamp function for ADD COLUMN
      Unverified
      7af4f706
    • Jeff Bruemmer's avatar
      update anchor links (#26070) · d54edd00
      Jeff Bruemmer authored
      Unverified
      d54edd00
    • adam-james's avatar
      Correctly Apply Column Formatting to Static Visualizations (#25493) · d1c61272
      adam-james authored
      * Column settings correct keys. Show Axes labels logic defaults 'true'
      
      These are two small changes related to static viz column formatting.
      
      The show axes labels logic change fixes a faulty assumption from an earlier PR where it was assumed that if
      :labels_enabled key was NOT present, then it was 'false'. The default however, is 'true', so the check had to change
      to reflect the frontend.
      
      Column settings have to be formatted so that the js code running in the backend looks in the right places. For
      example, `:number-format` must become `:number_format`. I move the function that does that change up so that it can be
      rolled up into `settings-from-column` to reduce the number of call sites for `update-col-for-js`.
      
      * Adjust number format fn to consider :decimal_digits of currencies
      
      * Simpler date-separator fix
      
      * Prevent non-temporal labels from being passed to datetime formatter
      
      * Table Temporal format columns are improved
      
      The column headers will now be properly looked up, the key format was incorrect before, causing the `:column_title`
      lookup to return nil, which ignores when a user changes the column name.
      
      Table columns with temporal formatting settings come in with `:unit` `:default`, so that case was added to the
      datetime formatter. Additionally, a helper function was added to pull the correct column viz settings to pass to the datetime.
      
      * Change some table rendering to look for '_' keys instead of '-'
      
      And also allow the field-id to take the form `[:field-id 10 nil]` OR `[:field-id 10]`
      
      * Added test utils for static-viz rendering. Docstrings for details.
      
      A big set of functions to enable cleaner static-viz rendering tests.
      
      Next push will be thorough table tests, and some other static viz cleanup
      
      * Add some viz-scenarios to help with table testing
      
      * Lint
      
      * Wrong map here
      
      * Currently unused bindings
      
      * Writing tests with render-test utils now, and catching new bugs
      
      * Use column's field ref to maybe get the column settings
      
      * Adjustments so that column-settings are properly grabbed
      
      * Number formatter no longer assumes decimal places
      
      The frontend will only display decimal places when strictly necessary OR when the column formatting dictates the
      number of decimal places to show.
      
      The number formatter on the backend made different assumptions about decimal places, so precision may be lost OR
      needlessly displayed.
      
      The formatter now checks the actual value passed in and adjusts the format string according to the decimal places
      present in the value, not just via column-settings, though the column settings will still take priority.
      
      * Shame on me. inline def is not needed here :/
      
      * Adjust number format tests to assert that now we match FE behavior
      
      * Really trying to get formatting buttoned up.
      
      * Matching frontend's default format behavior for numbers
      
      * Some tests failed because they assumed two decimal places always
      
      * Values less than one still round after 2 sig-figs
      
      * Column formatting on tables follows expected default behavior
      
      * Getting public-settings included in number and datetime formatters
      
      * Test for custom formatting public settings
      
      * Removed unused require.
      
      * address some PR feedback
      
      * Try to clarify the logic in the number formatter
      
      * Add some clarifying comments
      
      * Use the existing viz tools to help create field-refs properly
      
      * Cleaned up column-name impl. to use normed field-refs. Much nicer!
      
      * Remove some unnecessary tests, and add a negative scientific num
      
      * Simplify table columns test
      
      * Simplify and add more format settings to column-formatting test
      
      * Unused require
      
      * Another one
      
      * Fix format of column refs in test
      
      * Remove unused thread
      Unverified
      d1c61272
    • Braden Shepherdson's avatar
      Serdes v2: Refactor entire serdes flow to use real Toucan, not low level (#25981) · a9f44010
      Braden Shepherdson authored
      Previously, some automatic behavior was causing problems in serdes.
      (The worst example is generating an `entity_id` on insert while
      deserializing an entity we don't own - if deserialized again it would be
      duplicated.)
      
      A whole cascade of design choices fell out of this problem:
      - `insert!` would generate `entity_id`s, so use `simple-insert!`
      - But `simple-insert!` doesn't convert eg. MBQL `:definition` maps back
        into JSON strings
      - We got the maps as Clojure data and not JSON strings because `select`
        and `simple-select` both run `post-select` and parse the JSON.
      - So we ended up with a raw query on the select side and `simple-insert!`
        on the storage side.
      
      This change unwinds that whole stack, and instead uses a dynamic
      var to suppress the few pieces of `pre-insert` and `pre-update` logic
      that causes problems.
      
      The end result is much cleaner, and much more consistent with the rest
      of Metabase's backend logic.
      Unverified
      a9f44010
    • Natalie's avatar
      docs - update email (#26056) · 760eb6d9
      Natalie authored
      Unverified
      760eb6d9
    • Cal Herries's avatar
      Fix back navigation (#26049) · 8d7f1753
      Cal Herries authored
      Unverified
      8d7f1753
    • Natalie's avatar
      docs - update aws elastic beanstalk (#26042) · cfa280e1
      Natalie authored
      Unverified
      cfa280e1
    • Tim Macdonald's avatar
      Ensure static visualizations with multiple cards are rendered (#25811) · 6c226a69
      Tim Macdonald authored
      * Ensure static visualizations with multiple cards are rendered
      
      [Fixes #21559]
      
      * [static viz] Render multiple scalars as a bar chart
      Unverified
      6c226a69
  4. Oct 20, 2022
  5. Oct 19, 2022
  6. Oct 18, 2022
    • Gustavo Saiani's avatar
    • Noah Moss's avatar
      initial fix (#25991) · 9e43d98f
      Noah Moss authored
      Unverified
      9e43d98f
    • Cal Herries's avatar
      Open the data reference when the cursor moves to a card tag in the native query editor (#25801) · c46de9a2
      Cal Herries authored
      * Refactor DataReference
      
      * Don't show card tags in the sidebar
      
      * Refactor template tag regexes
      
      * Add models to DatabaseSchemasPane
      
      * Restyle SchemaPane
      
      * Anton's suggestions
      
      * Rename show to onItemClick
      
      * Remove another ternary
      
      * Add openDataReferenceAtCard action, but not opening the sidebar yet
      
      * Fix useEffect static analysis
      
      * Types
      
      * Get opening the sidebar at a model working, at the expense of model autocomplete
      
      * Change icon of ModelPane if it's a question
      
      * Tidy
      
      * Refactor DatabasePane
      
      * Move onItemClick and back callback logic to redux actions
      
      * Add e2e test
      
      * Add back autocomplete
      
      * Load question tables correctly
      
      * Restore autocomplete
      
      * Fix card tag autocomplete on the first character of the tag name
      
      * Update template tag editor visibility test to not test for card tags
      
      * Fix tests for cardIdFromTagName
      
      * Tidy tag editor
      
      * Typo
      
      * Fix undefined error
      
      * Tidy NativeQueryEditor.cardTagId
      
      * Tidy nextCompleters
      
      * Tidy
      
      * Update unit tests for tag editor visibility
      
      * Tidy
      
      * Tidy
      
      * Add test for models
      
      * Check fields have loaded before displaying table fields
      
      * Tidy
      
      * Rename ModelPane to QuestionPane
      
      * Rename card to question
      
      * Remove unused files
      
      * Tidy
      
      * Fix renaming
      
      * Fix renaming
      
      * Make variable template tag names non-empty
      
      * Fix test flakiness
      
      * Tidy
      
      * Tidy
      
      * Remove is_saved_questions check
      
      * Rename
      
      * Replace setDataReferenceStack action with selector and fix test
      
      * Initialize dataReferenceStack to null as an "uninitialized" state to fix the back button
      
      * Update e2e tests
      
      * Update e2e tests
      
      * Replace long ternaries
      Unverified
      c46de9a2
    • Anton Kulyk's avatar
      Disable data apps (#25977) · cef29d6d
      Anton Kulyk authored
      * Remove list and object viz from viz options
      
      * Hide apps and actions from new item menu
      
      * Remove UI for enabling writeback per-database
      
      * Disable app routes
      
      * Disable action routes
      
      * Disable model detail page
      
      * Don't list apps in the navbar
      
      * Hardcode dashboard's `isDataApp` to `false`
      
      * Remove "page" from custom destination options
      Unverified
      cef29d6d
    • Dalton's avatar
      Move more card/parameter value utils to metabase-lib (#25937) · dded4879
      Dalton authored
      * move remapParameterValuesToTemplateTags
      
      * Move more parameter value utils to metabase-lib
      
      * Fix tests
      Unverified
      dded4879
    • Alexander Polyankin's avatar
  7. Oct 17, 2022
Loading