Reload inbox after merge + pull-to-refresh on PR content; bump to 3.4
Build iOS App / Build .ipa and publish source (push) Successful in 34s
Build iOS App / Build .ipa and publish source (push) Successful in 34s
- After a merge, the app returns to the landing page but the inbox kept stale PRs. Add an inboxNeedsRefresh flag set on merge success; InboxView reloads on next appearance when it's set. - The PR content (incl. the Checks/workflow tab) is now pull-to-refreshable via .refreshable on the content ScrollView, so workflow status can be updated by pulling down. Bump to 3.4.
This commit is contained in:
1 parent
a9d408ee85
commit
a045a36125
4 files changed
+19
-6
No files matched your search
@@ -70,8 +70,9 @@ struct PRDetailView: View {
|
||||
}
|
||||
.sheet(isPresented: $showMergeSheet) {
|
||||
MergeSheet(vm: vm) {
|
||||
// Merge succeeded: show the success animation, then pop back
|
||||
// to the landing page.
|
||||
// Merge succeeded: mark the inbox for refresh, show the success
|
||||
// animation, then pop back to the landing page.
|
||||
SettingsStore.shared.inboxNeedsRefresh = true
|
||||
withAnimation(.spring(response: 0.4, dampingFraction: 0.6)) {
|
||||
showMergeSuccess = true
|
||||
}
|
||||
@@ -114,6 +115,7 @@ struct PRDetailView: View {
|
||||
case .none: OverviewTab(pr: pr)
|
||||
}
|
||||
}
|
||||
.refreshable { await vm.load() }
|
||||
|
||||
bottomTabBar
|
||||
}
|
||||
|
||||
@@ -33,6 +33,9 @@ final class SettingsStore: ObservableObject {
|
||||
didSet { UserDefaults.standard.set(monitoredRepos,
|
||||
forKey: "monitoredRepos") }
|
||||
}
|
||||
/// Set when a PR is merged, so the inbox knows to reload on next appearance
|
||||
/// (otherwise a just-merged PR could linger as stale in the list).
|
||||
@Published var inboxNeedsRefresh = false
|
||||
|
||||
private init() {
|
||||
instanceURL = KeychainHelper.get("instanceURL") ?? "https://gitea.bueso.eu"
|
||||
|
||||
@@ -60,6 +60,14 @@ struct InboxView: View {
|
||||
didInitialLoad = true
|
||||
await vm.load()
|
||||
}
|
||||
.onAppear {
|
||||
// After a merge, the inbox must reload so the just-merged PR
|
||||
// doesn't linger as stale in the list.
|
||||
if SettingsStore.shared.inboxNeedsRefresh {
|
||||
SettingsStore.shared.inboxNeedsRefresh = false
|
||||
Task { await vm.load() }
|
||||
}
|
||||
}
|
||||
.alert("Error", isPresented: .init(
|
||||
get: { vm.errorMessage != nil },
|
||||
set: { if !$0 { vm.errorMessage = nil } })) {
|
||||
|
||||
Reference in new issue
Block a user