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.

Sign up for GitHub

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

Jump to bottom

Fixes when using aggregates inside Spread Embedded Resources #3693

Merged
merged 4 commits into from
Aug 21, 2024
Merged

Fixes when using aggregates inside Spread Embedded Resources #3693

merged 4 commits into from
Aug 21, 2024

Commits on Aug 21, 2024

  1. fix: prevent spread embed to use aggregates when disabled

    @laurenceisla
    laurenceisla committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    bbba6a5 View commit details
    Browse the repository at this point in the history

  2. fix: a nested spread embedding now correctly groups by the fields of … 

    …its top parent relationship
    @laurenceisla
    laurenceisla committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    9807e60 View commit details
    Browse the repository at this point in the history

  3. refactor: simplify functions in the "hoist from selected fields" process

    @laurenceisla
    laurenceisla committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    63e6168 View commit details
    Browse the repository at this point in the history

  4. fix: spread embeds failing when using the "count()" aggregate without… 

    … a field - @laurenceisla

- Fixed "column reference <col> is ambiguous" error when selecting "?select=...table(col,count())"
- Fixed "column <json_aggregate>.<alias> does not exist" error when selecting "?select=...table(aias:count())"
    @laurenceisla
    laurenceisla committed Aug 21, 2024
    Configuration menu
    Copy the full SHA
    e44fbdd View commit details
    Browse the repository at this point in the history