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

🐛 Omit DataType=Platform when provided DataType does not include Platform #889

Merged
merged 1 commit into from
Oct 30, 2023

Conversation

jeremyf
Copy link
Contributor

@jeremyf jeremyf commented Oct 30, 2023

Prior to this commit, if the query parameters included only non-platform data_type (e.g. article) then we would continue to show the platform data type node.

With this commit, we skip showing that element if we have been provided explicit data_type(s) and none of them are Platform.

Related to:

@jeremyf jeremyf force-pushed the i686-omit-data-type-platform-when-data-type-specified branch from deb01f1 to 8c06db9 Compare October 30, 2023 15:43
…form

Prior to this commit, if the query parameters included only non-platform
data_type (e.g. article) then we would continue to show the platform
data type node.

With this commit, we skip showing that element if we have been provided
explicit data_type(s) and none of them are Platform.

Related to:

- #686 (comment)
@jeremyf jeremyf force-pushed the i686-omit-data-type-platform-when-data-type-specified branch from 8c06db9 to e7372eb Compare October 30, 2023 21:26
@jeremyf jeremyf merged commit d1805bf into main Oct 30, 2023
2 checks passed
@jeremyf jeremyf deleted the i686-omit-data-type-platform-when-data-type-specified branch October 30, 2023 21:26
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.

2 participants