feat: add loading state for app filters and update getRows logic

[AbhishekA1509]

Description

This pull request enhances the DevtronAppListContainer component to better handle loading states when application filters are being fetched. The main improvement is ensuring that the app list table does not attempt to load or display data while filter responses are still loading, which helps prevent inconsistent or empty data from being shown to users.

Loading state management improvements:

• The getRows function now checks the appFiltersResponseLoading flag and returns an empty result with zero rows if filters are still loading, preventing unnecessary data fetches and UI flicker.
• The dependency array for the getRows callback was updated to include appFiltersResponseLoading, ensuring the effect is recalculated when the loading state changes.
• The Table component now receives the appFiltersResponseLoading flag as its loading prop, so the table displays a loading state while filters are being fetched.

Prop handling:

• The DevtronAppList component now accepts the appFiltersResponseLoading prop, allowing it to respond to the filter loading state.

Fixes https://github.com/devtron-labs/sprint-tasks/issues/2865

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• Test A
• Test B

Checklist:

• The title of the PR states what changed and the related issues number (used for the release note).
• Does this PR require documentation updates?
• I've updated documentation as required by this PR.
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas

[github-actions]

Some linked issues are invalid. Please update the issue links:\nIssue # in is not found or invalid (HTTP }404).\n

[Copilot]

Pull request overview

This PR improves the Devtron app list table behavior while app-filter data (environments/namespaces) is still loading, preventing premature/undesired app-list fetching and showing a table-level loading state during filter fetch.

Changes:

• Gate getRows to return an empty result while appFiltersResponseLoading is true.
• Update getRows hook dependencies to include appFiltersResponseLoading.
• Pass appFiltersResponseLoading to the Table component via its loading prop.

Comments suppressed due to low confidence (1)

src/components/app/list/DevtronAppListContainer.tsx:167

• updateDataSyncing(false) won’t run if getAppList(...) rejects (network error) or is aborted via the passed signal, which can leave the parent isDataSyncing state stuck. Wrap the request in try/finally (and optionally handle abort separately) so syncing is always reset, even on errors/cancellation.

            updateDataSyncing(true)
            const data = await getAppList(getDevtronAppListPayload(filterConfig, environmentList, namespaceList), {
                signal,
            })
            updateDataSyncing(false)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud