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

Br fix datarequirements #1256

Merged
merged 3 commits into from
Oct 25, 2023
Merged

Br fix datarequirements #1256

merged 3 commits into from
Oct 25, 2023

Conversation

brynrhodes
Copy link
Member

This PR fixes several issues related to data requirements analysis, including #1202 and #1203

brynrhodes and others added 3 commits October 23, 2023 16:11
… property access

#1202: Fixed property access not being reported correctly when it occurred after the initial query context for a retrieve
#1202: Fixed data requirements not being reported correctly when a query source was itself a nested query
#1202: Fixed duplicate data requirements being reported for property access not directly related to a retrieve
#1202: Fixed enableResultTypes not working in some cases
@JPercival JPercival merged commit c53d2ce into master Oct 25, 2023
2 checks passed
@JPercival JPercival deleted the br-fix-datarequirements branch October 25, 2023 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants