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

Disallow mailto redirect #29406

Merged
merged 2 commits into from Jun 28, 2019
Merged

Disallow mailto redirect #29406

merged 2 commits into from Jun 28, 2019

Conversation

epeach
Copy link

@epeach epeach commented Jun 28, 2019

Students were using the openUrl block with 'mailto' link. We don't want to make it easy to facilitate students contacting each other this way, so we adding a third option to the redirect dialog that details that mailto uris are not supported.

@ajpal ajpal self-requested a review June 28, 2019 16:27
Copy link
Contributor

@ajpal ajpal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@epeach epeach merged commit aebd9e7 into staging Jun 28, 2019
@epeach epeach deleted the disallow-mailto-redirect branch July 2, 2019 20:50
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