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

Fix error thrown by nested keyFields: ["a", ["b", "c"], "d"] syntax when some of the fields are aliased #8643

Merged
merged 4 commits into from Aug 13, 2021

Conversation

benjamn
Copy link
Member

@benjamn benjamn commented Aug 12, 2021

Following @habovh's steps from #8639, expecting failure for the first commit.

I'll push more commits to fix the problem soon.

@benjamn benjamn changed the title Failing regression test for issue #8639 Fix error thrown by nested keyFields: ["a", "b", ["c", "d"], "e"] syntax when some of the fields are aliased Aug 12, 2021
@benjamn benjamn marked this pull request as ready for review August 13, 2021 00:00
@benjamn benjamn changed the title Fix error thrown by nested keyFields: ["a", "b", ["c", "d"], "e"] syntax when some of the fields are aliased Fix error thrown by nested keyFields: ["a", ["b", "c"], "d"] syntax when some of the fields are aliased Aug 13, 2021
Copy link
Member

@hwillson hwillson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great - thanks @benjamn!

@benjamn benjamn merged commit 1d5664c into main Aug 13, 2021
@benjamn benjamn deleted the issue-8639-nested-keyFields-problem branch August 13, 2021 22:00
@benjamn benjamn linked an issue Aug 16, 2021 that may be closed by this pull request
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Nested keyfields fails when using alias
2 participants