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

Added 'rest' and 'b2' repo support. Also added some minimal validation of mandatory repo fields #76

Merged
merged 2 commits into from
May 2, 2022

Conversation

quantum-byte
Copy link
Contributor

@quantum-byte quantum-byte commented May 1, 2022

  • Adds 'rest-server' protocol support
  • Adds 'b2' (Backblaze) protocol support
  • Adds some minimal better error handling for repo creation/editing
  • Downgrades com.android.tools.build:gradle to 7.0.4 since IntelliJ does not support it yet.
    • I can revert it but it makes my development slightly harder 😄
  • Does some things with the gradle wrapper but the version stays the same (7.2)
    • Not sure what happened there but i needed to do a ./gradlew wrapper --gradle-version 7.2 to get it to build for me.

@quantum-byte
Copy link
Contributor Author

Hi @lhns, i found this repo at the beginning of the year while looking for android backup solution with restic.
I looked nice but it did not support the Rest Server protocol yet which i wanted to use.
That is why i had a look and tried to work on it in my spare time.

I do have experience with Kotlin but this is my first Android project so please have an extra detailed look if I did anything which should not be done on Android.

Let me know what needs to be improved/changed before you can merge it. Then i ll have a look.

@lhns
Copy link
Owner

lhns commented May 2, 2022

Hi! Thank you very much for your work on this. I wanted to tackle the rest-server support for a while but didn't have any time yet so I am very happy to see it working with this PR.

@lhns lhns merged commit 1dd973b into lhns:main May 2, 2022
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.

2 participants