Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refresh button for source rows in Status page #6175

Merged
merged 57 commits into from
Feb 4, 2025
Merged
Changes from 1 commit
Commits
Show all changes
57 commits
Select commit Hold shift + click to select a range
2a8574f
new cell to refresh source in status tab
lovincyrus Nov 26, 2024
11378f3
move refresh sources to higher order
lovincyrus Nov 26, 2024
8babbc3
wip
lovincyrus Nov 26, 2024
263604a
add refresh confirm dialog
lovincyrus Nov 27, 2024
b878b9d
wip, fixme
lovincyrus Nov 27, 2024
227fd70
refetch allResources on row reload click
lovincyrus Dec 3, 2024
93b491c
force a new query using refetchKey for clean data
lovincyrus Dec 4, 2024
90d942e
add tooltip to menu item
lovincyrus Dec 4, 2024
e4e7b96
lint
lovincyrus Dec 4, 2024
04e0364
feedback, custom polling to avoid stale data
lovincyrus Dec 4, 2024
0dad749
error handling
lovincyrus Dec 4, 2024
4228a22
tsc whitelist
lovincyrus Dec 9, 2024
b56fbe0
Revert "tsc whitelist"
lovincyrus Dec 10, 2024
893dc80
add refresh trigger to new files
lovincyrus Dec 10, 2024
48a6fb7
stop polling when reconcile error
lovincyrus Dec 10, 2024
9d638f3
only stop polling when all resources are in idle status
lovincyrus Dec 11, 2024
0ecf036
rebase fix
lovincyrus Jan 24, 2025
6ac47e1
reserved space for actions in table
lovincyrus Jan 25, 2025
ca0fe8b
color loading spinner, loading type in notification
lovincyrus Jan 25, 2025
c43f6d6
confirmation dialog for single resource
lovincyrus Jan 25, 2025
eba66df
improve notifications based on diff resource status
lovincyrus Jan 25, 2025
82c56ba
lint
lovincyrus Jan 27, 2025
8298fed
refactor onError, feedback
lovincyrus Jan 27, 2025
c5ffad5
message fix
lovincyrus Jan 27, 2025
8efd19f
customizable timeout notification, feedback
lovincyrus Jan 27, 2025
5801459
use persisted, remove x from loading notification
lovincyrus Jan 27, 2025
82e29eb
proper notification clean up with id
lovincyrus Jan 28, 2025
7b998c5
hoist timeout to constants
lovincyrus Jan 28, 2025
1199ff6
max backoff
lovincyrus Jan 28, 2025
ab02acf
use resources, fetch interval
lovincyrus Jan 28, 2025
4ba771f
remove timestamp
lovincyrus Jan 28, 2025
98e3b7a
polling for refresh all
lovincyrus Jan 28, 2025
8d02b61
lint
lovincyrus Jan 28, 2025
973f7d6
readd loading notification, type check
lovincyrus Jan 28, 2025
53c7a6d
type check
lovincyrus Jan 28, 2025
a6a5a05
use is fetching for refreshing, max poll
lovincyrus Jan 29, 2025
83fbce7
handle navigate away case
lovincyrus Jan 29, 2025
39e930e
fix
lovincyrus Jan 29, 2025
ac8e223
optional type, clean up
lovincyrus Jan 29, 2025
afe8f95
fix
lovincyrus Jan 29, 2025
1d31e1e
polish navigate away and back notification
lovincyrus Jan 29, 2025
0c85b01
add refresh trigger to user facing resource kinds
lovincyrus Jan 30, 2025
2e0c8cf
add clear-all-notifications event
lovincyrus Jan 30, 2025
c00fcb3
feedback
lovincyrus Jan 30, 2025
f660ac3
lint
lovincyrus Jan 30, 2025
8609468
reset
lovincyrus Jan 30, 2025
25527d4
make refresh call async
lovincyrus Jan 30, 2025
a73d269
just right
lovincyrus Jan 30, 2025
08406c7
backoff strat, error handling
lovincyrus Jan 30, 2025
6bbf94c
move to pure function for refetch interval
lovincyrus Jan 31, 2025
4321669
clean up, clean up, clean up
lovincyrus Jan 31, 2025
78d9b08
lint
lovincyrus Jan 31, 2025
b0811b8
reverts
lovincyrus Feb 3, 2025
3ef4c92
Revert "reverts"
lovincyrus Feb 3, 2025
40f2da7
update refetch interval
lovincyrus Feb 4, 2025
18e4b93
revalidate in refresh all
lovincyrus Feb 4, 2025
2a5f7d0
fix action cell sizing, compact
lovincyrus Feb 4, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix
lovincyrus committed Jan 29, 2025
commit 39e930e6a4a25d9b6724f85ef0f0a1268832eddd
Original file line number Diff line number Diff line change
@@ -161,6 +161,7 @@
if (isPollingEnabled) {
eventBus.emit("notification", {
type: "clear-all",
message: "",
});
}
});
@@ -180,12 +181,13 @@
});
}

// Flag to track when user revisits the page
// Track when user navigates away and revisits the page
onMount(() => {
isLoaded = true;
});

// Continue polling if user revisits the page and there are non-idle resources
// Continue polling if user navigates away and revisits the page
// and there are non-idle resources
$: if (isLoaded && $resources.data && !isPollingEnabled) {
const hasNonIdleResources = $resources.data.resources?.some(
isResourceReconciling,
2 changes: 1 addition & 1 deletion web-common/src/lib/event-bus/events.ts
Original file line number Diff line number Diff line change
@@ -5,7 +5,7 @@ interface Link {

export interface NotificationMessage {
type: "default" | "success" | "loading" | "error" | "clear-all";
message?: string;
message: string;
detail?: string;
link?: Link;
options?: NotificationOptions;