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

Backup and restore #39

Merged
merged 27 commits into from
Oct 24, 2020
Merged

Backup and restore #39

merged 27 commits into from
Oct 24, 2020

Conversation

NeoTheThird
Copy link
Member

After coordination with the original author, i'm taking over #28 to prevent further divergence and merge conflicts.

Thank you to @Alain94W for his original work!

Added backup function, devicestatus (recovery,device,bootloader) info, floder or partition size info, customecommand execution (not only adb for the backup). Started to write the restore function but not tested yet, working on the GUI first, do not use the restore function now.

@NeoTheThird NeoTheThird marked this pull request as draft October 22, 2020 18:03
@NeoTheThird NeoTheThird marked this pull request as ready for review October 23, 2020 18:46
@NeoTheThird
Copy link
Member Author

I'm not adding tests for the backup functions yet themselves, as i want them to be refactored when we move to streams. I hope to be able to start the work on that soon, so it would be a waste of time now, seen as how it's an experimental feature anyways.

@NeoTheThird NeoTheThird requested a review from Flohack74 October 23, 2020 18:48
@NeoTheThird
Copy link
Member Author

also cf ubports/ubports-installer#141

@NeoTheThird NeoTheThird requested a review from mariogrip October 24, 2020 09:14
@NeoTheThird NeoTheThird mentioned this pull request Oct 24, 2020
@NeoTheThird NeoTheThird merged commit e19c113 into master Oct 24, 2020
@NeoTheThird NeoTheThird deleted the Alain94W-backup-restore branch October 24, 2020 10:14
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.

3 participants