frontend/src/pages/GeneralSettings/DataConnectors/Connectors/Github/index.jsx

Instead of accessing the properties of the data object multiple times, you can destructure the data object once and use the variables directly. This will make the code cleaner and easier to read.
Create Issue
See the diff
Checkout the fix

    const { files, author, repo, branch, destination } = data;
    showToast(
      `${files} ${pluralize("file", files)} collected from ${
        author
      }/${repo}:${branch}. Output folder is ${destination}.`,
      "success",
      { clear: true }
    );

git fetch origin && git checkout -b ReviewBot/Impro-q83og7c origin/ReviewBot/Impro-q83og7c

The handleSubmit function is recreated every time the component re-renders. This can lead to unnecessary re-renders and performance issues. By using the useCallback hook, you can ensure that the function is only recreated when its dependencies change.
Create Issue
See the diff
Checkout the fix

    const handleSubmit = useCallback(async (e) => {
      // existing code
    }, [ignores]);

git fetch origin && git checkout -b ReviewBot/Impro-m62oewp origin/ReviewBot/Impro-m62oewp

frontend/src/components/SettingsSidebar/index.jsx

Using a ternary operator can make the code more concise and easier to read. It's a good practice to use a ternary operator when you want to assign a value to a variable based on a condition.
Create Issue
See the diff
Checkout the fix

    const handleBg = () => {
      showSidebar ? setTimeout(() => setShowBgOverlay(true), 300) : setShowBgOverlay(false);
    }

git fetch origin && git checkout -b ReviewBot/Impro-1ov377k origin/ReviewBot/Impro-1ov377k