-
Notifications
You must be signed in to change notification settings - Fork 129
fix: buildQueryOptions error #3132
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: buildQueryOptions error #3132
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
commit: |
Graphite Automations"Test" took an action on this PR • (10/08/25)1 assignee was added to this PR based on Kacper Wojciechowski's automation. |
c3abe3e to
f784f10
Compare
8662f05 to
dbd3c7a
Compare
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
f784f10 to
366f8f7
Compare
dbd3c7a to
d55922f
Compare
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
20b6f95 to
17a4dbf
Compare
e0096df to
0ab7942
Compare
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
17a4dbf to
2a323ff
Compare
0ab7942 to
0d1d02f
Compare
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
2a323ff to
dacdaef
Compare
0d1d02f to
6d7b5f2
Compare
|
Claude encountered an error —— View job I'll analyze this and get back to you. |
Merge activity
|
Closes FRONT-838 ### TL;DR Hide the "Instances" section in the sidebar when the data provider doesn't support query options. ### What changed? Added a check to verify if the data provider has the `buildsQueryOptions` capability before rendering the "Instances" section in both the Subnav and CloudSidebarContent components. This prevents displaying the Instances section when it's not applicable. ### How to test? 1. Connect to a data provider that doesn't support query options 2. Verify that the "Instances" section is not displayed in the sidebar 3. Connect to a data provider that does support query options 4. Verify that the "Instances" section appears correctly ### Why make this change? Some data providers don't support query options, making the "Instances" section irrelevant for those providers. This change improves the UI by only showing relevant navigation options based on the capabilities of the connected data provider.

Closes FRONT-838
TL;DR
Hide the "Instances" section in the sidebar when the data provider doesn't support query options.
What changed?
Added a check to verify if the data provider has the
buildsQueryOptionscapability before rendering the "Instances" section in both the Subnav and CloudSidebarContent components. This prevents displaying the Instances section when it's not applicable.How to test?
Why make this change?
Some data providers don't support query options, making the "Instances" section irrelevant for those providers. This change improves the UI by only showing relevant navigation options based on the capabilities of the connected data provider.