feat: return if data is fetching

[YounixM]

Important

Add check for isFetchingDeploymentsData in useEffect in CustomDomainSettings.tsx to prevent execution during data fetching.

• Behavior:
  • Adds a check for isFetchingDeploymentsData in useEffect in CustomDomainSettings.tsx to prevent execution when data is fetching.
  • Updates dependencies of useEffect to include isFetchingDeploymentsData.

^{This description was created by for c25f182. It will automatically update as commits are pushed.}

[github-actions]

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to c25f182 in 11 seconds

More details

• Looked at 29 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. frontend/src/container/CustomDomainSettings/CustomDomainSettings.tsx:109

• Draft comment:
  Consider adding setHosts and setCustomDomainDetails to the dependency array of the useEffect hook to ensure all dependencies are covered.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The useEffect hook is missing a dependency for setHosts and setCustomDomainDetails. These functions are used within the effect, and while they are stable in this context, it's a good practice to include them in the dependency array to avoid potential issues in the future if they change.

Workflow ID: wflow_D3ESNrDNHIDlAJSN

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[github-actions]

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

[request-info]

We would appreciate it if you could provide us with more info about this issue/pr!

[github-actions]

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>