Skip to content
This repository has been archived by the owner on Apr 1, 2021. It is now read-only.

Add intermediate solution to Algorithm-Roman-Numeral-Converter #1151

Closed
wants to merge 4 commits into from

Conversation

mrthnmn
Copy link

@mrthnmn mrthnmn commented Jun 16, 2016

A second (intermediate) solution has been added.

@ghost ghost self-assigned this Jun 16, 2016
@ghost ghost assigned Rafase282 Jun 16, 2016
@ghost ghost added this to the Update Algorithms milestone Jun 16, 2016
@@ -68,6 +68,70 @@ convertToRoman(36);
- [For Loops](JS-For-Loop)
- [While Loops](JS-while-Loop)

## :intermediate: Intermediate Code Solution:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:intermediate: :sunflower:

@Rafase282
Copy link
Member

Please add a section to our articles related to your solution.

@Rafase282 Rafase282 changed the title Mrthnmn-Algorithm-Roman-Numeral-Converter.md Add intermediate solution to Algorithm-Roman-Numeral-Converter Jun 16, 2016
@mrthnmn
Copy link
Author

mrthnmn commented Jun 16, 2016

I'm afraid I'm not sure what you want. I thought I was editing the existing
wiki article on the Roman Numeral Converter Algorithm challenge. Do you
want a more-thorough explanation of the code I added or a completely new
wiki article?

On Thu, Jun 16, 2016 at 12:18 PM, Rafael J. Rodriguez <
notifications@github.com> wrote:

Please add a section to our articles related to your solution.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#1151 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AQKaMFz1fpkkEt55GlFjrDTruTymn_pPks5qMXdGgaJpZM4I3gRi
.

@Rafase282
Copy link
Member

Please take a look here #1094

We have a template for solutions, and you adding a new one also requires a section with related links. I was asking to add it and use links form our wiki if possible. Sorry for not being clear before.

- Add links after both code explanations
- Add a sunflower image in front of "Intermediate Code Solution"
- Clean up code in Intermediate Code Solution
- Add "@mrthnmn" to credits

- [Roman Numerals](http://www.mathsisfun.com/roman-numerals.html)
- [Array.splice()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/splice)
- Array.indexOf()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/indexOf)
Copy link

@ghost ghost Jun 17, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Array.indexOf()] [Array.indexOf()]
Change this so it links to our internal article: JS-Array-Prototype-IndexOf.

@Rafase282
Copy link
Member

Please try to fix the conflicts on your branch. Feel free to come to the wiki chat if you need help.


- [Roman Numerals](http://www.mathsisfun.com/roman-numerals.html)
- [Array.splice()](https://github.com/FreeCodeCamp/wiki/blob/master/JS-Array-Prototype-Splice.md)
- Array.indexOf()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/indexOf)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-- Array.indexOf()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/indexOf)
+- [Array.indexOf()](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/indexOf)

- [For Loops](JS-For-Loop)
- [While Loops](JS-while-Loop)
- [For Loops](https://github.com/FreeCodeCamp/wiki/blob/master/JS-For-Loop.md)
- [While Loops](https://github.com/FreeCodeCamp/wiki/blob/master/JS-While-Loop.md)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-- [While Loops](https://github.com/FreeCodeCamp/wiki/blob/master/JS-While-Loop.md)
+- [While Loops](JS-While-Loop)

@ghost
Copy link

ghost commented Jun 18, 2016

@mrthnmn, please resolve merge conflicts and implement suggestions 😉

@mrthnmn
Copy link
Author

mrthnmn commented Jun 20, 2016

I will do on Monday. I wasn't able to on Saturday because I was behind a
corporate proxy.

On Sat, Jun 18, 2016 at 2:27 PM, Jonathan notifications@github.com wrote:

@mrthnmn https://github.com/mrthnmn, please resolve merge conflicts and
implement suggestions 😉


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#1151 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AQKaMHK8qlsdYSabI93g0N0J4BV1uwMcks5qNDiEgaJpZM4I3gRi
.

@alayek
Copy link
Member

alayek commented Jun 20, 2016

@mrthnmn this branch has conflicts. You might need to rebase the branch.

@mrthnmn
Copy link
Author

mrthnmn commented Jun 20, 2016

Attempt to resolve conflicts

@mrthnmn mrthnmn closed this Jun 20, 2016
@BerkeleyTrue BerkeleyTrue removed the QA label Jun 20, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants