-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Download limit for file link share #11563
base: master
Are you sure you want to change the base?
Conversation
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/11563.apk |
@@ -0,0 +1,81 @@ | |||
/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please move those RemoteOperation to android-library.
If possible, also add real end to end tests, else I can help you with them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please grant me push access to android-library?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tobiasKaminsky I was writing IT for the download limit but I am facing issue in executing them in android-library.
- For parsing the test cases are running fine as there is no use of AbstractIT class.
- But for Operations (Get, Post, Delete) we need AbstractIT to be extended and for this the test cases are not executing. I tried existing test cases of NC but same for them.
Is there some configuration is required to execute them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code moved to android-library: nextcloud/android-library#1123
I asked nextcloud/files_downloadlimit#173 to have a capability, as probably not that many server will have this enabled. |
700c84f
to
f21db86
Compare
master-IT test failed, but no output was generated. Maybe a preliminary stage failed. |
test-Unit test failed, but no output was generated. Maybe a preliminary stage failed. |
blue-Light-Screenshot test failed, but no output was generated. Maybe a preliminary stage failed. |
Implemented download limit for files where share type is Link.
Showing the download count once the download limit is specified.
Conditions to show download limit:
Required library PR: nextcloud/android-library#1123
Requires the implementation of: https://github.com/nextcloud/files_downloadlimit