Skip to content
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

added Threema Libre to notification preferences #29

Closed
wants to merge 1 commit into from

Conversation

thigg
Copy link

@thigg thigg commented Oct 14, 2022

@pvuorela
Copy link
Contributor

The notification file is not getting used anymore for the current android support, only for old android4 on couple of devices. I'm not sure does it make sense to keep adding content there.

@bernhardreiter
Copy link

@pvuorela on my Android App Support 10 I needed to make the change documented at https://forum.sailfishos.org/t/4-0-1-48-notifications-from-android-apps-not-shown/4872/21 .

So is there a different defect or is this not the latest Android support?

@pvuorela
Copy link
Contributor

So this system was disabled here for the new android support with commit 3035cb8. Reason being that the Android notification integration has been getting improvements, and if there are shortcomings that would be the correct place to fix them. On the upcoming release it should automatically use these chat/chat_exists feedbacks if the Android app uses the CATEGORY_MESSAGE.

@bernhardreiter
Copy link

@pvuorela so I understand that a better solution is not in the current release, but will be in the next release. Cool! Thanks for the explanation!

@pvuorela
Copy link
Contributor

Yes, exactly.

@thigg
Copy link
Author

thigg commented Oct 25, 2022

This sounds like this pr wants to be closed

@thigg thigg closed this Oct 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants