fix #10640: the interface fails to be queried,/v1/graphql#10705
fix #10640: the interface fails to be queried,/v1/graphql#10705david-ourlegup wants to merge 1 commit intohasura:masterfrom
Conversation
The schema registry notification color hook has been updated to improve performance. A check was added to skip API calls if running in CE mode or if the schema registry host is not configured. This change also includes a modification that skips the query completely when the schema registry is unavailable, and returns a loading state instead. These enhancements aim to optimize resource usage and provide better user experience.
|
Thanks @david-ourlegup for submitting this PR to improve the schema registry notification color hook! These optimizations look very useful - particularly skipping API calls in CE mode and when the schema registry host isn't configured, and the improved handling when the registry is unavailable. |
|
Thanks again @david-ourlegup for your PR contribution. To help me better understand the changes: Could you please elaborate on what specific problem you encountered that led to these optimizations? I'd like to understand the exact issue you're trying to solve. Your insights will help ensure we properly test and validate the improvements before merging. Looking forward to your response! |
The schema registry notification color hook has been updated to improve performance. A check was added to skip API calls if running in CE mode or if the schema registry host is not configured. This change also includes a modification that skips the query completely when the schema registry is unavailable, and returns a loading state instead. These enhancements aim to optimize resource usage and provide better user experience.