Fix remaining follow request notif problems

The problems were:

• mouseover text was wrong for reject
• approving follow request didn't mark that notification as seen
• approving/denying follow requests from the separate follow request page didn't change notifications accordingly

all been fixed, closes #823 (closed) again

To support marking single notifications as seen, the pleroma notification-read endpoint has been adopted, and the qvitter one finally retired.

changed the description

mentioned in issue #823 (closed)

added 1 commit

• 75519223 - mark single notifs as seen properly on server

Compare with previous version

changed the description

@shpuld, LGTM except for one floating issue (is not always reproducible):

Clicking on "x" in notification no longer removes the notification (until the page is reloaded).

I haven't been able to reproduce the issue, but I'll look around a bit

ok I was able to reproduce it now, not sure about the steps, I'll try to fix it

seems that when this happens, it's because of BE returning 403 for the dismiss request

Screen_Recording_2020-05-02_at_11.23.46

Not sure of the steps either. Somehow it started working for a few times, then stoppped working again. Will check BE responses.

Rejecting a follow request automatically dismisses notification for it. So, FE doesn't need to send dismiss request after reject request.

Huh interesting, good to know. So I guess we'll just need to remove it in frontend.

added 1 commit

• 92ccaa97 - don't dismiss a rejected follow request on server

Compare with previous version

Alright, I think it should be fixed now. thanks for the debugging help @i1t !

Works good, thanks for the fixes!

Looks good!

mentioned in commit 5f90b6a3

merged

mentioned in merge request pleroma!2461 (closed)

mentioned in merge request pleroma!2462 (merged)