Skip to content
This repository has been archived by the owner on Dec 12, 2020. It is now read-only.

Dropbox v2 - Beta #509

Open
wants to merge 25 commits into
base: master
Choose a base branch
from
Open

Dropbox v2 - Beta #509

wants to merge 25 commits into from

Conversation

beavel
Copy link

@beavel beavel commented Jul 6, 2018

This PR represents a working version against Dropbox v2, which should complete #481. I've only tested it on Android 7.0 at the moment, but was able to get it to load my todo, add tasks, and sync completions. The browsing for directory also works as tested in a limited fashion. This may be a bit of rough hack as I'm not fully versed in the event cycle of Android apps, but I tried to make as limited code changes as necessary to replace the functionality. Not sure what the next steps are from here, but I can build successfully and run by creating a dropbox.xml and commenting out the resValue for dropbox in the debug gradle configuration.

What I know needs to be handled/decided on:

  • How to handle upgrades - I've only installed fresh on an old phone. Anything special that needs to be done.
  • Signing the release version
  • Dropbox API key to use - I created one for personal use/testing.
  • Re-enable proguard if needed

beavel added 25 commits July 1, 2018 11:34
Comment out packages not currently dependent on to avoid multidex issue
Compiles, installs, but fails to run on Android 7.0
@dae3
Copy link
Contributor

dae3 commented Aug 9, 2018

@beavel this looks good. I independently started the same work before noticing your PR. We have made almost exactly the same changes but yours is complete. I haven't fixed the authentication code yet.

I don't have the old version to upgrade from but from considering the code I think it should just be updating the SharedPreferences to purge the old authentication details and forcing re-authentication.

I can't help with signing or a more official Dropbox API key - @evanp do you know who has ownership of the signing key and Dropbox API key used by the currently published app?

It's sideloaded on my phone now to get some real-world testing.

@beavel
Copy link
Author

beavel commented Aug 11, 2018

@dae3 I'm glad this is working for you! Also its good to know I was on the right track with the changes since we had the same ones.

I haven't installed this on my main phone as I still have my old credentials there and that will be my one opportunity to work through the upgrade process flow with cleaning out credentials. If there is no movement no this PR in the near future, I might just reinstall and sideload to get my own real-world testing in as well. Going to be patient and hope that we can get this pushed out to other users though.

@dae3
Copy link
Contributor

dae3 commented Sep 2, 2018

Hey @beavel I'm seeing weird corruption in todo.txt after editing (add, change, complete) on the app. It looks like the unicode byte order is being reversed at the point of the edit, making the file a mess in desktop editors (gvim and notepad).

I've just spent an hour repairing my todo.txt with a hex editor :-( . I'll try to reproduce later in the week.

@herrold
Copy link

herrold commented Sep 25, 2018

@evanp

You appear to have commits ... the Android client is still broken a year later as Dropbox turned down the v1 API. Could you please apply this diff ?

Thank you

@beavel
Copy link
Author

beavel commented Oct 6, 2018

@dae3 Did you get a chance to look into the file corruption issue? I haven't had time recently to look and was just wondering if you got a chance to look a little further. Still haven't installed it on my main phone, but I might here soon if there is no further activity on this PR.

@dae3
Copy link
Contributor

dae3 commented Oct 7, 2018 via email

@dae3
Copy link
Contributor

dae3 commented Oct 21, 2018

@beavel I've had no file corruption for 2 weeks now, after carefully manually cleaning up my todo.txt file back to 8 bit ASCII.

I can't explain how it occurred but it hasn't returned, despite daily use of the app and desktop text editors to maintain the

@kvithayathil
Copy link

@dae3 i was about to work on a project upgrade to android studio 3.3.1 but I saw that you've already done it. Look forward to your work getting merged! 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants