-
-
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
Dont dismiss notification after each upload/download #7949
Conversation
a2dc1db
to
0195070
Compare
Blue-Light-IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/7949-Screenshot-Blue-Light-21-19 |
@newhinton tested and works 👍 Can you rebase it and have a look at spotbugs? There seem to be some more warnings coming with this PR. |
0195070
to
1dde934
Compare
❗ REBASED ❗ ...since Spotbugs results are deleted (housekeepign) so trigger run again based on latest master @newhinton |
1dde934
to
f282f41
Compare
Spotbugs should be resolved with the latest commit, let's see what CI is thinking of it :) |
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/7949.apk |
89a866d
to
7d9db68
Compare
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/7949.apk |
master-IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/763-IT-master-15-19 |
Blue-Dark-Screenshot test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/7949-Screenshot-Blue-Dark-15-19 |
White-Dark-Screenshot test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/7949-Screenshot-White-Dark-15-19 |
Blue-Light-Screenshot test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/7949-Screenshot-Blue-Light-15-32 |
White-Light-Screenshot test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/7949-Screenshot-White-Light-15-28 |
Signed-off-by: Felix Nüsse <felix.nuesse@t-online.de>
…ndomness (just creating Notification IDs) Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>
7d9db68
to
9e2920b
Compare
CodacyLint
SpotBugs (new)
SpotBugs (master)
|
stable-IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/819-IT-stable-11-17 |
This PR aims to 'group' notifications for transfer-progress together so that they are not obstrusive. Instead of recreating each notification (and therefore create distracting effects in the notifcation area of android) the existing notification is reused and just updated.
Also no additional 'success'-message is beeing shown, only errors are displayed. Each error has it's own id, so that they are preserved after the upload. This applies to 'normal' up and downloads. Instant-Upload acts slightly different and cannot use this change to the full extend.
Closes #7651