bito-code-review[bot] commented on PR #44101: URL: https://github.com/apache/superset/pull/44101#issuecomment-5859740051
<!-- Bito Reply --> The flagged issue regarding `commands.getCommand` being checked only during rendering is correct. To resolve this, you should move the command check into a `useEffect` hook or a state-management pattern that listens for command registration events, ensuring the menu updates dynamically when commands are added or removed. Since the file `superset-frontend/src/features/home/RightMenu.tsx` is not present in the provided PR diff, I cannot implement the fix directly. Please verify if the file is part of this PR or if the path provided in the comment is accurate. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
