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.

Sign up for GitHub

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

Jump to bottom

feat: snapshot status indicator #1123

Merged
merged 19 commits into from
Sep 18, 2023
Merged

feat: snapshot status indicator #1123

merged 19 commits into from
Sep 18, 2023

Conversation

1emu
Copy link
Collaborator

@1emu 1emu commented Jul 25, 2023 •
edited
Loading

Closes #1249

image image

Mobile view with menu open

image image image

@yemel yemel had a problem deploying to governance-pr-1123 July 25, 2023 03:33 Failure
@1emu 1emu force-pushed the feat/snapshot-status-indicator branch from 191e7f1 to 03dd3f2 Compare July 25, 2023 03:37
@1emu 1emu temporarily deployed to governance-pr-1123 July 25, 2023 03:38 Inactive
@1emu 1emu force-pushed the feat/snapshot-status-indicator branch from 03dd3f2 to c6131e0 Compare July 27, 2023 04:29
@yemel yemel temporarily deployed to governance-pr-1123 July 27, 2023 04:29 Inactive
@yemel yemel temporarily deployed to governance-pr-1123 July 27, 2023 05:07 Inactive
@1emu 1emu force-pushed the feat/snapshot-status-indicator branch from 1033ecd to 066c681 Compare September 12, 2023 20:06
@1emu 1emu marked this pull request as ready for review September 12, 2023 20:16
@1emu 1emu changed the title feat: snapshot ping POC feat: snapshot status indicator Sep 12, 2023
@1emu 1emu requested a review from ncomerci September 13, 2023 17:07
@1emu 1emu temporarily deployed to governance-pr-1123 September 13, 2023 21:31 Inactive
@1emu 1emu temporarily deployed to governance-pr-1123 September 13, 2023 21:40 Inactive
andyesp
andyesp reviewed Sep 14, 2023
View reviewed changes
src/clients/SnapshotApi.ts Outdated Show resolved Hide resolved
@1emu 1emu temporarily deployed to governance-pr-1123 September 14, 2023 15:16 Inactive
@1emu 1emu temporarily deployed to governance-pr-1123 September 14, 2023 18:40 Inactive
@andyesp andyesp merged commit 4582aff into master Sep 18, 2023
1 check passed
@andyesp andyesp deleted the feat/snapshot-status-indicator branch September 18, 2023 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ncomerci ncomerci ncomerci approved these changes

@andyesp andyesp andyesp approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Snapshot error state
4 participants
@1emu @andyesp @ncomerci @yemel