Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Destroy follow requests on exhausted delivery attempts #21958

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ClearlyClaire
Copy link
Contributor

@ClearlyClaire ClearlyClaire commented Dec 2, 2022

In rare cases, it is possible a follow request never reaches the server of the person that one is trying to follow.

Currently, that failure case is indistinguishable from other failure cases as far as the end-user is concerned, but that failure case can be detected by Mastodon.

In such a case, it might be more useful to delete the follow request than show it as pending, although I am not quite sure how this could be perceived by the user attempting to follow. Ideally, the error should be recorded and displayed to the user, but that requires changes to the API, storage and clients.

Also related: #21956, #21957

@jsoref
Copy link
Contributor

jsoref commented Dec 4, 2022

Couldn't you trigger a notification to the user? I get notifications for Follows, Replies, and Favorites. Do you really need to define a new notification type, or could you just use an existing one?

@ClearlyClaire
Copy link
Contributor Author

Couldn't you trigger a notification to the user? I get notifications for Follows, Replies, and Favorites. Do you really need to define a new notification type, or could you just use an existing one?

No, we can't use an existing notification type for that, we'd need a new one.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2023

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2023

This pull request has resolved merge conflicts and is ready for review.

@ClearlyClaire
Copy link
Contributor Author

In such a case, it might be more useful to delete the follow request than show it as pending, although I am not quite sure how this could be perceived by the user attempting to follow. Ideally, the error should be recorded and displayed to the user, but that requires changes to the API, storage and clients.

We probably do want to make those changes, and that should probably be through adding a column to follow requests, as well as change the response serializer to embed this information.

@github-actions
Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants