Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Skip to content

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

👪 ❤️ https://gitlab.e.foundation/groups/e/-/wikis/contribute/development-processes/code-review-guidelines

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
  • 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

Merge request reports

Loading