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

add restore support as alternate mode #63

Merged
merged 1 commit into from
Sep 8, 2023
Merged

add restore support as alternate mode #63

merged 1 commit into from
Sep 8, 2023

Conversation

scicco
Copy link

@scicco scicco commented Jun 12, 2023

Hello, this will address #59

@scicco scicco mentioned this pull request Jun 12, 2023
Closed
@maxisam maxisam merged commit 9ee80d6 into maxisam:main Sep 8, 2023
@maxisam
Copy link
Owner

maxisam commented Sep 8, 2023

Thank you for your contribution. Sorry for taking so long to take care this.

@maxisam
Copy link
Owner

maxisam commented Sep 9, 2023 •
edited
Loading

@scicco sorry, in the end, I changed my mind on implementation. I think it makes more sense to be an on demand action than setting in the plan. Because we shouldn't change the plan and rerun the app just for restore. I have implemented it in #69

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@scicco @maxisam