[notifications] send notification filters in API request #10

New Issue

Open

opened 2022-06-23 10:21:53 +00:00 by floatingghost · 0 comments

floatingghost commented

2022-06-23 10:21:53 +00:00

Owner

the masto API supports filtering notifications on the request: https://docs.joinmastodon.org/methods/notifications/

the FE does not send any filters on notif type in the request

and instead runs a filter on all notifications client-side

this leads to small filters only having 1 or 2 entries because we only fetched n notifications

we should modify this behaviour to ask the server to filter for us

the masto API supports filtering notifications on the request: https://docs.joinmastodon.org/methods/notifications/ the FE [does not send any filters on notif type in the request](https://akkoma.dev/AkkomaGang/pleroma-fe/src/branch/develop/src/services/notifications_fetcher/notifications_fetcher.service.js#L9) and instead runs [a filter on all notifications client-side](https://akkoma.dev/AkkomaGang/pleroma-fe/src/branch/develop/src/components/notifications/notifications.js#L57) this leads to small filters only having 1 or 2 entries because we only fetched n notifications we should modify this behaviour to ask the server to filter for us