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

FIX: User website allows new TLDs #4819

Merged
merged 1 commit into from Apr 18, 2017
Merged

FIX: User website allows new TLDs #4819

merged 1 commit into from Apr 18, 2017

Conversation

@xfalcox
Copy link
Member

xfalcox commented Apr 18, 2017

No description provided.

@mention-bot

This comment has been minimized.

Copy link

mention-bot commented Apr 18, 2017

Thanks @xfalcox for your pull request 👍. By analyzing the blame information on this pull request, I identified @techAPJ and @nlalonde to be potential reviewers.

@discoursebot

This comment has been minimized.

Copy link

discoursebot commented Apr 18, 2017

You've signed the CLA, xfalcox. Thank you! This pull request is ready for review.

@xfalcox xfalcox requested a review from techAPJ Apr 18, 2017
@techAPJ

This comment has been minimized.

Copy link
Member

techAPJ commented Apr 18, 2017

Looks good to me! 👍

@techAPJ techAPJ merged commit fa1b6da into master Apr 18, 2017
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@ZogStriP ZogStriP deleted the website-validation branch Apr 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants
You can’t perform that action at this time.