openmobilehub / android-omh-storage

https://www.openmobilehub.org
Apache License 2.0
4 stars 1 forks source link

fix/OMHD-512: OneDrive update fix #71

Closed maciekBudzinski closed 2 months ago

maciekBudzinski commented 2 months ago

Summary

This PR provides workaround for updating the file. Read comments in file to get more context.

Demo

https://github.com/user-attachments/assets/e6327b35-168c-4e8a-b0f2-666585b1055e

Checklist:

Closes: OMHD-512 Closes: OMHD-464

damiann5f commented 2 months ago

I think this PR is more related to OMHD-515 than OMHD-464 - a core issue in the OMHD-464 ticket was related to the access token, but during testing of a fix, the issue from OMHD-515 popped out, and it was assumed it was a new bug caused by the fix to the access token issue).

Alternatively add in the description that it close both of those tasks?

maciekBudzinski commented 2 months ago

@DamianNowak5f Good catch, updated title and description. Won't rename branch as it's gonna to be removed after merge. Hope that it works for you.