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

Transfer ownership to invited user results in HTTP 500 #2640

Closed
2 tasks done
Popescu-PfeifferMarc opened this issue Feb 2, 2023 · 4 comments · Fixed by #2651
Closed
2 tasks done

Transfer ownership to invited user results in HTTP 500 #2640

Popescu-PfeifferMarc opened this issue Feb 2, 2023 · 4 comments · Fixed by #2651

Comments

@Popescu-PfeifferMarc
Copy link

Past Issues Searched

  • I have searched open and closed issues to make sure that the bug has not yet been reported

Issue is a Bug Report

  • This is a bug report and not a feature request, nor asking for self-hosted support

Using official Plausible Cloud hosting or self-hosting?

Self-hosting

Describe the bug

Transferring ownership to another user, who hasn't accepted the join request results in HTTP 500.

Expected behavior

An error message is shown, e.g. "this user hasn't accepted the join request".

Screenshots

No response

Environment

No response

@ruslandoga ruslandoga self-assigned this Feb 2, 2023
@ruslandoga
Copy link
Contributor

ruslandoga commented Feb 2, 2023

👋 @Popescu-PfeifferMarc

What Plausible version are you running?

A similar issue #2455 has been fixed in v1.5.

@metmarkosaric
Copy link
Contributor

thanks for reporting @Popescu-PfeifferMarc! this happens on the cloud too when an invitation is sent to the same email address that's already on the pending invitation list. if you remove the person from the list of pending invitations first, you can invite them again without any problems. rather than showing a random error we should have a user friendly message instead saying something like:

You're trying to invite an email address that already has a pending invitation. Please remove the pending invitation before trying again

@Popescu-PfeifferMarc
Copy link
Author

The issue was resolved by the invited user accepting the invite. I just wanted to suggest adding a more descriptive error message to help future users running into the same problem

@Popescu-PfeifferMarc
Copy link
Author

👋 @Popescu-PfeifferMarc

What Plausible version are you running?

A similar issue #2455 has been fixed in v1.5.

It should be the latest release version of plausible (v1.5.1)

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

Successfully merging a pull request may close this issue.

3 participants