Skip to content

fix typo and add comma#3986

Merged
bugron merged 1 commit intofreeCodeCamp:stagingfrom
MartinDelille:patch-1
Oct 30, 2015
Merged

fix typo and add comma#3986
bugron merged 1 commit intofreeCodeCamp:stagingfrom
MartinDelille:patch-1

Conversation

@MartinDelille
Copy link
Copy Markdown
Contributor

latitiude => latitude

@arialaurel
Copy link
Copy Markdown
Member

Thanks for the PR - I definitely think some better formatting of this data in the example would be an improvement. However, it looks like without using HTML entities for the <br> tag it won't show up in the page. Could you replace the <br> with &lt;br&gt;? 😄

@arialaurel arialaurel mentioned this pull request Oct 29, 2015
@BerkeleyTrue BerkeleyTrue added the status: blocked In a transient & temporary hold. label Oct 29, 2015
@MartinDelille
Copy link
Copy Markdown
Contributor Author

done

@MartinDelille
Copy link
Copy Markdown
Contributor Author

Tell me if you want me to squash the commit!

@arialaurel
Copy link
Copy Markdown
Member

That would be awesome if you could squash them!

latitiude => latitude
add space and <br>
@MartinDelille
Copy link
Copy Markdown
Contributor Author

done

@bugron
Copy link
Copy Markdown
Contributor

bugron commented Oct 30, 2015

@MartinDelille thanks for the PR! Now the result looks better, verified and merged.
I recommend you to read our Guidelines for Contributing. Also next time write in the description of your PR something like closes #4010 of fix #4010 to auto close the issue which your PR is fixing (if there is an open issue, of course).

bugron added a commit that referenced this pull request Oct 30, 2015
@bugron bugron merged commit 69c5139 into freeCodeCamp:staging Oct 30, 2015
@bugron bugron removed the status: blocked In a transient & temporary hold. label Oct 30, 2015
@bugron bugron mentioned this pull request Oct 30, 2015
@MartinDelille MartinDelille deleted the patch-1 branch October 30, 2015 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants