[DRAFT] Implements file upload progress

[ariedov]

This is the most basic implementation of the file upload progress feature.

It contains:

1. DB update and migration
2. Flows between the dao-repository-viewmodel
3. A TextView to display the updates on the screen

[github-actions]

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

[mahibi]

providesFileUploadsDao

[ariedov]

Hey! Thanks for taking a look. I did build the branch and kinda got it to work. Will look into this.

[mahibi]

Hey! Thanks for taking a look. I did build the branch and kinda got it to work. Will look into this.

ok that's interesting. Could you share some setup details? Android Studio version and Gradle JDK (Preferences -> Build -> Build Tools -> Gradle -> Gradle JDK)

Does it still work when you clean the build beforehand (Build - Clean project)?

[tobiasKaminsky]

Took me ~1h, but finally found it.
But stracktrace, debug, logs, all were not helpful.
In the end I opened all changed files one by one to check for IDE reporting problems.
And then it turned out that ChatActivity had 2x companion object…

[mahibi]

Thanks @tobiasKaminsky
now it builds again.
@ariedov i just tried if it builds and saw that currently the pollng polling is broken. This is because handleUploadsFromDb is blocking at the moment. To solve, for example it's content should be wrapped by

scope.launch {

}

Also, during the first test i did not see progress for the text view (it was null 0.0).
I can have a closer look next week. Otherwise just let me know if you want to change anything beforehand.
Thanks again for your efforts!

[ariedov]

Thank you very much for fixing the build and spending some effort on this! I am extremely out of time, unfortunately, at the moment, so I cannot promise to spend much time on this anytime soon.

Sorry for leaving this one in a half-baked state. I'll do my best to try to pick up the slack as soon as I can.

[mahibi]

Thank you very much for fixing the build and spending some effort on this! I am extremely out of time, unfortunately, at the moment, so I cannot promise to spend much time on this anytime soon.

Sorry for leaving this one in a half-baked state. I'll do my best to try to pick up the slack as soon as I can.

All fine @ariedov , we appreciate your efforts!
If at any time you feel like you are not able to continue with the PR at all, just let us know if we should take over.
Otherwise we are happy to help!

[mahibi]

Hi @ariedov
do you plan pick this up again or should me or me colleagues take over?

[ariedov]

Hey @mahibi , sorry for the silence. Please feel free to take over.

[mahibi]

Hey @mahibi , sorry for the silence. Please feel free to take over.

No worries. Thanks to let me know and thanks again for your efforts!
I rebased for now. We will pick it up when time allows!
If you change your mind and would like to continue just let us know.

[github-actions]

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/5317.apk



To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

[github-actions]

Codacy

Lint

Type	master	PR
Warnings	99	99
Errors	0	0

SpotBugs

Category	Base	New
Bad practice	6	6
Correctness	10	10
Dodgy code	54	54
Internationalization	3	3
Malicious code vulnerability	3	3
Performance	4	4
Security	1	1
Total	81	81