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

Added link about Roman Numeral to MDNLinks section. #2598

Merged
merged 1 commit into from
Sep 9, 2015
Merged

Added link about Roman Numeral to MDNLinks section. #2598

merged 1 commit into from
Sep 9, 2015

Conversation

suhussai
Copy link
Contributor

Added link mentioned in Waypoint description about
Roman Numerals to MDNLinks in order to make it more apparent.
Affects Waypoint 3 in Intermediate Algorithm Scripting.
http://www.freecodecamp.com/challenges/bonfire-roman-numeral-converter
Closes #2557

@suhussai suhussai changed the title Added link about Roman Numeral to MDNLinks section Added link about Roman Numeral to MDNLinks section, closes #2557. Aug 24, 2015
@BerkeleyTrue
Copy link
Contributor

Please remove issue numbers from title

see Pull-Request-Contribute

@suhussai suhussai changed the title Added link about Roman Numeral to MDNLinks section, closes #2557. Added link about Roman Numeral to MDNLinks section. Aug 30, 2015
@suhussai
Copy link
Contributor Author

Its updated. Thanks for the link, I'll look through it.

@@ -113,6 +113,7 @@
"convert(36);"
],
"MDNlinks": [
"Roman Numerals",

This comment was marked as off-topic.

Added link mentioned in Waypoint description about
Roman Numerals to MDNLinks in order to make it more apparent.
Affects Waypoint 3 in Intermediate Algorithm Scripting.
http://www.freecodecamp.com/challenges/bonfire-roman-numeral-converter
closes #2557.
@suhussai
Copy link
Contributor Author

suhussai commented Sep 9, 2015

Fixed the indent issue. Thanks for the tip.

@ahstro
Copy link
Member

ahstro commented Sep 9, 2015

Looks good, thanks! :)

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.

3 participants