Add permanent hint when battery optimization is not ignored

[mahibi]

Also add an option to not show the hint, when user doesn't want to ignore battery optimization.

🖼️ Screenshots

ignore battery optimization hint	ignore battery optimization dialog	Option to hide the hint

🏁 Checklist

• [ ] ⛑️ Tests (unit and/or integration) are included or not needed
• [ ] 🔖 Capability is checked or not needed
• [ ] 🔙 Backport requests are created or not needed: /backport to stable-xx.x
• [ ] 📅 Milestone is set
• [ ] 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

[mahibi]

/backport to stable-20.0

[mahibi]

Should it say "not disabled" or "not ignored"? I think the warning should say disabled?

i will change the warning so it will summarize if anything is wrong with notifications, not only battery optimization. So it will include if notification are disabled in general or if notification app is not installed on server.. So the wording of the hint will be different in the end.

[nickvergessen]

Can you update the screenshots?

[github-actions[bot]]

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



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

[github-actions[bot]]

Codacy

Lint

Type	master	PR
Warnings	94	158
Errors	132	76

SpotBugs

Category	Base	New
Bad practice	6	6
Correctness	16	16
Dodgy code	71	71
Internationalization	3	3
Malicious code vulnerability	3	3
Performance	4	4
Security	1	1
Total	104	104

Lint increased!