Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.x] [Vis Builder] Add field summary popovers #2797

Merged
merged 1 commit into from
Nov 4, 2022

Commits on Nov 4, 2022

  1. [Vis Builder] Add field summary popovers (#2682)

    * [Vis Builder] Add field summary popovers
    
    Much of the functionality was ported from `Discover`, but
    largely refactored.
    
    * Add utilities to get sampled hit summaries by field
    * Add popover summaries
    * Slight refactor of special `Count` pseudofield
    * Use observable subscription to update sampled hits
    
    Fixes #950
    
    Signed-off-by: Josh Romero <rmerqg@amazon.com>
    
    * [Vis Builder] Add additional unit tests
    
    Signed-off-by: Josh Romero <rmerqg@amazon.com>
    
    * [VisBuilder] Update naming of summary field components
    
    Signed-off-by: Josh Romero <rmerqg@amazon.com>
    
    * [VisBuilder] Avoid prop passing by extracting custom hooks
    
    - refactor meta field identification
    
    Signed-off-by: Josh Romero <rmerqg@amazon.com>
    
    * [VisBuilder] Add TODOs with issue links, fix test ID
    
    Restores previous test ID for count field button
    
    Signed-off-by: Josh Romero <rmerqg@amazon.com>
    
    Signed-off-by: Josh Romero <rmerqg@amazon.com>
    (cherry picked from commit 2e16de1)
    Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
    
    # Conflicts:
    #	CHANGELOG.md
    github-actions[bot] committed Nov 4, 2022
    Configuration menu
    Copy the full SHA
    bf1fc81 View commit details
    Browse the repository at this point in the history