3467 activate silently ntfy on push murena com
Description
ntfy as been added in /e/OS as the foundation.e.ntfy app since a few versions. But it is disabled by default, and it will use ntfy.sh server.
The Broadcasting notification project also includes hosting of an ntfy instance on push.murena.com domain (https://gitlab.e.foundation/groups/e/os/-/epics/39#note_701302).
- To make use of this infra as expected, we have to update the URL in foundation.e.ntfy code
- To have the broadcasting notification system working out of the box, we have to activate ntfy by default in /e/OS
- To avoid user deactivate it UnitifedPush provider without proper feedback, remove the possibility to disable it through UI
Screenshots
Technical details
Revert the /e/OS modifications made on ntfy which added the MainSettingsActivity, and its registration in settings app.
Tests
- Observe that the UnifiedPush entry isn't visible anymore in the Settings -> System screen
- Check that UnifiedPush is working (using direct messages in Pachli app for example, and lookng for "ntfy" logs in logcat, and that the push URI mention push.murena.com )
Issues
https://gitlab.e.foundation/e/os/backlog/-/issues/3467
10 commandments of code review
Summary by CodeRabbit
- New Features
- Background listener now restarts automatically when needed for instant subscriptions.
- Bug Fixes
- Foreground notification is cleared properly when the background service stops.
- UI/Style
- Notification icon updated: larger and consistently white for improved visibility.
- Changes
- Default server URL updated to https://push.murena.com.
- Refactor
- Removed the in-app EOS settings screen and its Android Settings entry to streamline the experience.
- Chores
- Cleaned up build repositories and unused dependencies.
Edited by Guillaume Jacquart