Skip to content

Conversation

@lawalalao
Copy link

@lawalalao lawalalao commented Nov 29, 2020

add solution link to 14th lines

Why:

What's being changed:

Check off the following:

  • All of the tests are passing.
  • I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
  • For content changes, I have reviewed the localization checklist
  • For content changes, I have reviewed the Content style guide for GitHub Docs.

add solution link to 14th lines
@welcome
Copy link

welcome bot commented Nov 29, 2020

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

@janiceilene
Copy link
Contributor

Thanks for opening a PR 👍 Please fill out the pull request template in the top comment in this pull request, then we'll get this up for review!

@lawalalao
Copy link
Author

done

@janiceilene
Copy link
Contributor

@lawalalao Can you answer the Why? and What's being changed? to help us understand why you want to add a solutions link? When I look at the staging link for the changes you've made, there isn't a new page listed.

What page are you looking to add a link to?

@github-actions
Copy link
Contributor

This PR is stale because it has been open 7 days with no activity and will be automatically closed in 3 days. To keep this PR open, update the PR by adding a comment or pushing a commit.

@github-actions github-actions bot added the stale There is no recent activity on this issue or pull request label Dec 12, 2020
@shati-patel shati-patel self-assigned this Dec 22, 2020
@shati-patel
Copy link
Contributor

Thank you again for taking the time to contribute, @lawalalao! 😃 I'm going to close this PR for now, as we don't have a solution file to link to. Feel free to re-open, or open a new PR if you have any further suggestions 📝

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale There is no recent activity on this issue or pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants