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 env variables link on TROUBLESHOOTING.md #801

Closed
wants to merge 1 commit into from
Closed

Fix env variables link on TROUBLESHOOTING.md #801

wants to merge 1 commit into from

Conversation

imagentleman
Copy link
Contributor

Fixes

Checklist

  • I have made a material change to the repo (functionality, testing, spelling, grammar)
  • I have read the [Contribution Guide] and my PR follows them.
  • I updated my branch with the master branch.
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation about the functionality in the appropriate .md file
  • I have added in line documentation to the code I modified

Short description of what this PR does:

  • Fixes the "environment variables" link on TROUBLESHOOTING.md, which points to a section that does not exist on the README.md. Changed it to point to the equivalent section on CONTRIBUTING.md.

If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.

@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label Oct 13, 2018
@reedsa
Copy link
Contributor

reedsa commented Oct 15, 2018

Duplicates work done in #789

@reedsa reedsa added status: duplicate duplicate issue and removed status: code review request requesting a community code review or review from Twilio labels Oct 15, 2018
@imagentleman
Copy link
Contributor Author

Closing since it's a duplicate.

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

Successfully merging this pull request may close these issues.

3 participants