Skip to content

Settings: notification tray limit (notificationTrayLimit) #35

Closed

Description

@Adron

Part of #31 — Settings & preferences parity — most of PATCH /api/user/update is unreachable from Android

The web's bell tray shows a configurable number of notifications (notificationTrayLimit).
Android's notifications feed uses a hard-coded page size and the push-poll caps its tray group
independently.

  • Expose the preference and have both the in-app notifications list and the system-tray grouping
    respect it.

Tests: PATCH round-trip; list honours the limit.


Filed from the parity review of feature/parity-wip @ 14d7795 against interlinedlist.com (OpenAPI /api/openapi.json, 233 paths) and the public help centre.

Activity

  1. added
    parityClosing a gap against interlinedlist.com
    P2Completeness / settings surface
    on Sep 15, 2026
  2. Adron commented on Sep 16, 2026

    @Adron
    MemberAuthor

    Implemented in the PR above (branch issue/35-notification-tray-limit). Closing here — review and any follow-up happens on the PR.

    Placement and range came from the docs: the web files this under View preferences, not under a notifications group (/help/settings lists it beside Messages per page, and its Notifications section points back at it), and the valid range is 10–40, default 20 — stated on /help/settings and corroborated by /help/api/notifications ("clamped to 10–40") and the live GET /api/user value.

    Both consumers now honour it, which was the substance of the issue: the in-app list stops using a hard-coded page size, and the system-tray grouping drops its independent MAX_INDIVIDUAL = 5 cap in favour of the account's limit.

    One behaviour change to be aware of: a batch of six notifications now posts six individually where it used to collapse into a summary. That is the intent, but worth knowing before it surprises anyone.

    Crossing the module boundary needed a second narrow accessor in :core:network alongside #19's — so this adds another reader to the ownership tangle tracked in #104.

  3. added a commit that references this issue on Sep 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2Completeness / settings surfacearea:notificationsNotifications & pusharea:settingsSettings & preferencesparityClosing a gap against interlinedlist.com

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions