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

Invitation emails appear sendable in the UI despite the mail setup being incomplete #11670

Closed
5 tasks done
miya opened this issue Dec 15, 2024 · 0 comments · Fixed by #11809
Closed
5 tasks done

Invitation emails appear sendable in the UI despite the mail setup being incomplete #11670

miya opened this issue Dec 15, 2024 · 0 comments · Fixed by #11809
Labels
💪 enhancement New feature or request

Comments

@miya
Copy link
Contributor

miya commented Dec 15, 2024

Self Checks

  • I have searched for existing issues search for existing issues, including closed ones.
  • I confirm that I am using English to submit this report (我已阅读并同意 Language Policy).
  • [FOR CHINESE USERS] 请务必使用英文提交 Issue,否则会被关闭。谢谢!:)
  • Please do not modify this template :) and fill in all the required fields.

1. Is this request related to a challenge you're experiencing? Tell me about your story.

Even when the mail setup is incomplete, the invitation email appears to be sendable.

However, this could be misleading, as clicking the invite button won't actually send the email, which might be unhelpful.

Since the invitation link generated after clicking the invite button can be shared through services like Discord, disabling the invite button might not be ideal.

As a solution, displaying a warning message on the Invite modal when the mail setup is incomplete would be a good approach.

Reference image

スクリーンショット 2024-12-15 21 22 55

2. Additional context or comments

No response

3. Can you help us with this feature?

  • I am interested in contributing to this feature.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💪 enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant