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

Update links throughout tool #71

Merged
merged 5 commits into from Sep 2, 2019

Conversation

@MattIPv4
Copy link
Member

commented Sep 2, 2019

This pull request makes the following changes:

  • Resolves #68 by deep linking to the correct host/record type for the propagation checker
  • Fixes #69 by ensuring the propagation checker URL opens in a new tab safely
  • Fixes #70 by creating a new external link component that opens in a new tab safely
  • Fixes an issue with the help icon that appears next to TTL being on a new line

@MattIPv4 MattIPv4 self-assigned this Sep 2, 2019

@JakeMakesStuff JakeMakesStuff changed the title 🚧 Update links throughout tool Update links throughout tool Sep 2, 2019

@MattIPv4 MattIPv4 merged commit 6f91587 into master Sep 2, 2019

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

@MattIPv4 MattIPv4 deleted the links branch Sep 2, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.