Skip to content

Conversation

Hubert-Szczepanski-SAP
Copy link
Contributor

What this PR does / why we need it:

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

@andreaskienle
Copy link
Contributor

andreaskienle commented Aug 11, 2025

I've noticed this loading state behavior isn't just limited to the graph - it's also affecting the provider config table, which briefly shows a loading state during data reloads.

SWR already differentiates between isLoading (for the initial fetch) and isValidating (for subsequent fetches like polling). It looks like our ⁠useProvidersConfigResource hook might be setting ⁠isLoading to ⁠true for every fetch operation which causes the problem.

Could you take a look at this? I think we could improve the user experience by updating ⁠useProvidersConfigResource to only return ⁠isLoading: true during the initial fetch, following SWR's pattern. Checking this in the Graph component doesn’t feel right.

@Hubert-Szczepanski-SAP Hubert-Szczepanski-SAP merged commit 8fc1c01 into main Aug 11, 2025
5 checks passed
@Hubert-Szczepanski-SAP Hubert-Szczepanski-SAP deleted the feat/managed-resource-graph-refresh-bug branch August 11, 2025 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants