jiebi/CodeConvo huggingface dataset:
Code change:
So the problem is that the “hint” where it tells you what you should’ve have coded it tells you that you should have “data” as key, when you code that and it doesn’t work it tells you you should have “myTaskArr” as key, but that’s wrong it should tell you “myTaskArr” as “value”. So I was trying to have 2 keys, and obviously it was always wrong. The hint should tell you that "myTaskArr" is a value, not a key as it currently says. ! Device: Desktop PC OS: Windows 10 Browser: Edge Version: [e.g. 22] 120.0.2210.121
Note: Due to PR adding one more step, earlier in this project, the relevant step in codebase is 49 now. Although that change did not yet reach the production at this time.

Related discussion:
text: assert.match(code, /localStorage.setItem(("|')data1/) ``` <del> Your `localStorage.setItem()` should have a key of `myTaskArr`. </del> <ins> Your `localStorage.setItem()` should have a value of `myTaskArr`. </ins> ```js assert.match(code, /localStorage.setItem(("|')data1,s*myTaskArr);?/)
Code change:
The description here is missing backticks for the number zero. Here is the new updated description. This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors. If you would like to contribute and have not read the contributors docs, please do so here: If you have any issues with contributing, be sure to join us on the , or on the Here is the correct file that needs to be updated. Note: Even though the file says step 62 instead of step 63 that is on the site, this is still the correct file that needs updating. We recently made changes to the ordering and those changes haven't shown up yet on the live site. Happy coding see explanation above see explanation above No response Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response
Step 145 has the same issue here Here is the file that needs to be updated here is the correct use of backticks for the number zero in the last sentence here

Related discussion:
text: Pass in only the first element of the `locations` array by adding `[0]` at the end of the variable. For example: `myFunction(arg[0]);`. <del> This is called <dfn>bracket notation</dfn>. Values in an array are accessed by index. Indices are numerical values and start at 0 - this is called zero-based indexing. `arg[0]` would be the first element in the `arg` array. </del> <ins> This is called <dfn>bracket notation</dfn>. Values in an array are accessed by index. Indices are numerical values and start at `0` - this is called zero-based indexing. `arg[0]` would be the first element in the `arg` array. </ins> # --hints--
Code change:
The description here is missing backticks for the number zero. Here is the new updated description. This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors. If you would like to contribute and have not read the contributors docs, please do so here: If you have any issues with contributing, be sure to join us on the , or on the Here is the correct file that needs to be updated. Note: Even though the file says step 62 instead of step 63 that is on the site, this is still the correct file that needs updating. We recently made changes to the ordering and those changes haven't shown up yet on the live site. Happy coding see explanation above see explanation above No response Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response
Step 145 has the same issue here Here is the file that needs to be updated here is the correct use of backticks for the number zero in the last sentence here

Related discussion:
text: The attack of the monster will be based on the monster's `level` and the player's `xp`. In the `getMonsterAttackValue` function, use `const` to create a variable called `hit`. Assign it the equation `(level * 5) - (Math.floor(Math.random() * xp));`. <del> This will set the monster's attack to five times their `level` minus a random number between 0 and the player's `xp`. </del> <ins> This will set the monster's attack to five times their `level` minus a random number between `0` and the player's `xp`. </ins> # --hints--
Code change:
Describe the bug Task 37 requires typing in one of the blanks. This can be problematic due to the letter, which might not be present in the used keyboard layout. Expected behavior I don't want to say to use completely different word for the fill the blank, as at some point there might be no other choice. There could be some kind way to ease up procuring of the letter in question, for example including it in the feedback, so it can be copied.
Hi I would love to work on this. If someone could point me in the right direction, that would be amazing!
Hi Thank you for being eager to contribute. This issue has not been triaged yet. Feel free to share your insights and ideas on the matter. However as far as the code contributing, take a look at the issues with label. Please make sure you read , we prioritize contributors following the instructions in our guides. Join us in or if you need help contributing, our moderators will guide you through this. Thank you and happy coding.
Yes, I think in both of those challenges we can update the "Don't forget the stress mark." in the feedback to: "Don't forget the stress mark ()."
I will open the issue to contribution with this as the solution

Related discussion:
text: ### --feedback-- <del> Places where you can buy and eat food. Don't forget the stress mark. </del> <ins> Places where you can buy and eat food. Don't forget the stress mark (`é`). </ins> # --scene--
Code change:
Describe the bug Task 37 requires typing in one of the blanks. This can be problematic due to the letter, which might not be present in the used keyboard layout. Expected behavior I don't want to say to use completely different word for the fill the blank, as at some point there might be no other choice. There could be some kind way to ease up procuring of the letter in question, for example including it in the feedback, so it can be copied.
Hi I would love to work on this. If someone could point me in the right direction, that would be amazing!
Hi Thank you for being eager to contribute. This issue has not been triaged yet. Feel free to share your insights and ideas on the matter. However as far as the code contributing, take a look at the issues with label. Please make sure you read , we prioritize contributors following the instructions in our guides. Join us in or if you need help contributing, our moderators will guide you through this. Thank you and happy coding.
Yes, I think in both of those challenges we can update the "Don't forget the stress mark." in the feedback to: "Don't forget the stress mark ()."
I will open the issue to contribution with this as the solution

Related discussion:
text: ### --feedback-- <del> Places to eat or drink. Don't forget the stress mark. </del> <ins> Places to eat or drink. Don't forget the stress mark (`é`). </ins> ---
Code change:
A camper on the forum had issues with passing step 37 of the pyramid project This was their code Their code is correct but the tests expect a certain order of The tests should be updated to expect either correct answer see explanation above see explanation above No response Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response
const character = "#"; const count = 8; const rows = []; for (let i = 0; i < count; i = i + 1) { (i); } let result = "" for (const row of rows) { result = + ; (result); } This worked for me, it took a while but realized I needed to swap the result with the row... This was helpful thank you
Thank you for this,I was really stuck

Related discussion:
text: You should use the concatenation operator on your `result` variable. ```js <del> assert.match(code, /results*+/); </del> <ins> assert.match(code, /(?:results*+|+s*result)/); </ins> ``` You should concatenate `row` to your `result` variable. ```js <del> assert.match(code, /results*+s*row/); </del> <ins> assert.match(code, /results*+s*row|rows*+s*result/); </ins> ``` You should assign the result of your concatenation back to the `result` variable. ```js <del> assert.match(code, /results*=s*results*+s*row;?/) </del> <ins> assert.match(code, /results*=s*(results*+s*row|rows*+s*result);?/); </ins> ``` # --seed--
Code change:
When working through the , it's expected from the user to create a method that internally uses a to make a sum, like the following: But when presented the following solution, making the reduce's callback sum the other way around, it doesn't pass: Because of the , the previous presented solution should work. No response Device: Laptop OS: macOS 13.6.6 Browser: Chrome Version: 124 No response
I agree that both versions should be accepted since they are valid solutions.

Related discussion:
text: ```js const afterCalculateTotal = code.split('calculateTotal')[1]; <del> assert.match(afterCalculateTotal, /(s*totals*,s*items*)s*=>s*totals*+s*item.price/); </del> <ins> assert.match(afterCalculateTotal, /(s*totals*,s*items*)s*=>s*(?:totals*+s*item.price|item.prices*+s*total)/); </ins> ``` Your `reduce` call should have an initial value of `0`. ```js const afterCalculateTotal = code.split('calculateTotal')[1]; <del> assert.match(afterCalculateTotal, /thiss*.s*itemss*.s*reduces*(s*(s*totals*,s*items*)s*=>s*totals*+s*item.prices*,s*0s*)/); </del> <ins> assert.match(afterCalculateTotal, /thiss*.s*itemss*.s*reduces*(s*(s*totals*,s*items*)s*=>s*(?:totals*+s*item.price|item.prices*+s*total)s*,s*0s*)/); </ins> ``` # --seed--
Code change:
Currently on step number 5 there's a note that reads Note that when reassigning a variable that has already been declared, you do not use the let keyword. As it stands this note is incorrect and unnecessary. No code, just instructions re-wording. The note could be changed to: Note: You can only re-assign a variables that has been declared with Or we can simply remove it altogether instead. <img width="1217" alt="Screen Shot 2024-05-02 at 10 31 38" src=""- Device: [Laptop] OS: [MacOS 12.7.4] Browser: [Chrome] Version: [124.0.6367.91] No response
The note is correct. I think you are interpertering it wrong? It is saying does not need to be used when re-assinging the variable with a new value. edit: It is not saying that you can use any kind of keyword like or
I understand what's it's trying to convey, but it's written in a confusing way. Here's the text directly copied into ChatGPT for some additional context: <img width="820" alt="Screen Shot 2024-05-02 at 11 01 54" src=""
Text refers to potential situation when both assignments are using , ie: Depending on where exactly both are used, this can result in . - - x = 10; There's no on this line. It's one of the cases when chat doesn't know what it's talking about.
Hi thank you for creating the issue. As Sem pointed out, we don't think the note should be updated to because the new version communicates different information from the original. I think the confusion came from the word "declared". The word isn't needed since "reassigning" already implies that the variable has been declared, so I'd propose changing the sentence a little: cc'ing for input, since you're much more experienced with this kind of change than I am
Thanks for the input your proposal addresses the issue I presented
That change works for me. Let's open it up for first timers This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors. If you would like to contribute and have not read the contributors docs, please do so here: If you have any issues with contributing, be sure to join us on the , or on the js let programmer = "Naomi"; programmer = "CamperChan"; ` Happy coding

Related discussion:
text: programmer = "CamperChan"; ``` <del> Note that when reassigning a variable that has already been declared, you do **not** use the `let` keyword. </del> <ins> Note that when reassigning a variable, you do **not** use the `let` keyword again. </ins> After your `console.log`, assign the value `"World"` to your `character` variable.
Code change:
There is a typo at the following step: It should present the text: Device: Laptop OS: Windows 11 Browser: Brave Version: 1.

Related discussion:
text: transform: translateY(-50%); } <ins> .landing-skill-icon { color: #215f1e; font-size: 150px; } .black-text { color: #333; font-weight: 400; font-size: 40px; } .font-awesome-padding { margin-top: 45px; margin-bottom: 20px; } </ins> .background-svg { width: 220px; height: 220px;
Code change:
There is a typo at the following step: It should present the text: Device: Laptop OS: Windows 11 Browser: Brave Version: 1.

Related discussion:
text: background-position: center; } <ins> .testimonial-image { border-radius: 5px; height: 200px; width: 200px; } .testimonial-copy { font-size: 20px; text-align: center; @media (min-width: 991px) and (max-width: 1199px) { height: 120px; } @media (min-width: 1200px) { height: 90px; } } </ins> //uncomment this to see the dimensions of all elements outlined in red //* { //  border-color: red;
Code change:
There is a typo at the following step: It should present the text: Device: Laptop OS: Windows 11 Browser: Brave Version: 1.

Related discussion:
text: extends layout block content <del> .hidden-xs a(href='https://github.com/freecodecamp/freecodecamp') img(style='position: absolute; top: 40; right: 0; border: 0; margin-top: -30px;', src='https://camo.githubusercontent.com/e7bbb0521b397edbd5fe43e7f760759336b5e05f/68747470733a2f2f73332e616d617a6f6e6177732e636f6d2f6769746875622f726962626f6e732f666f726b6d655f72696768745f677265656e5f3030373230302e706e67', alt='Fork me on GitHub', data-canonical-src='https://s3.amazonaws.com/github/ribbons/forkme_right_green_007200.png') </del> .jumbotron .text-center h1.hug-top Code with Us
Code change:
There is a typo at the following step: It should present the text: Device: Laptop OS: Windows 11 Browser: Brave Version: 1.

Related discussion:
text: br br a.btn.nonprofit-cta.btn-success(href="/nonprofits") I'm with a nonprofit and want help coding something <del> include partials/about include partials/faq </del>  No newline at end of file <ins> .big-break h2 Campers you'll hang out with: .row .col-xs-12.col-sm-12.col-md-4 img.img-responsive.testimonial-image.img-center(src="https://s3.amazonaws.com/freecodecamp/testimonial-jen.jpg", alt="@jenthebest's testimonial image") .testimonial-copy Getting back on track with Free Code Camp and committing to a new career in 2015! h3 - @jenthebest .col-xs-12.col-sm-12.col-md-4 img.img-responsive.testimonial-image.img-center(src="https://s3.amazonaws.com/freecodecamp/testimonial-tate.jpg", alt="@TateThurston's testimonial image") .testimonial-copy Just built my company's website with skills I've learned from Free Code Camp! h3 - @TateThurston .col-xs-12.col-sm-12.col-md-4 img.img-responsive.testimonial-image.img-center(src="https://s3.amazonaws.com/freecodecamp/testimonial-cynthia.jpg", alt="@cynthialanel's testimonial image") .testimonial-copy  I'm currently working through Free Code Camp to improve my JavaScript. The community is very welcoming! h3 - @cynthialanel .big-break h2 Skills you'll learn: .text-center .col-xs-12.col-sm-12.col-md-3 .landing-skill-icon.ion-social-html5 .black-text HTML5 .col-xs-12.col-sm-12.col-md-3 .landing-skill-icon.ion-social-css3 .black-text CSS3 .col-xs-12.col-sm-12.col-md-3 .landing-skill-icon.ion-social-javascript .black-text JavaScript .col-xs-12.col-sm-12.col-md-3 .landing-skill-icon.fa.fa-database.font-awesome-padding .black-text Databases .col-xs-12.col-sm-12.col-md-3 .landing-skill-icon.ion-social-chrome .black-text DevTools .col-xs-12.col-sm-12.col-md-3 .landing-skill-icon.ion-social-nodejs .black-text Node.js .col-xs-12.col-sm-12.col-md-3 .landing-skill-icon.ion-social-angular .black-text Angular.js .col-xs-12.col-sm-12.col-md-3 .landing-skill-icon.ion-ios-loop-strong .black-text Agile .big-break h2 &nbsp; h2 Fast facts about our community: h3.col-xs-offset-0.col-sm-offset-1.col-md-offset-2 ul.text-left li.ion-code &nbsp; We're 100% free and open source li.ion-code &nbsp; We're thousands of professionals who are learning to code li.ion-code &nbsp; We're building projects for dozens of nonprofits li.ion-code &nbsp; We share one goal: to boost our careers with code .big-break a.btn.btn-cta.signup-btn(href="/login") Start learning to code (it's free) </ins>  No newline at end of file
Code change:
In the first paragraph, on the 2nd phrase, you can read: . Rectify to the following: No response . No response
Thanks for opening this issue. This looks something that can be fixed by "first time" code contributors to this repository. Here are the files that you should be looking at to work on a fix: List of files: Please make sure you read , we prioritize contributors following the instructions in our guides. Join us in or if you need help contributing, our moderators will guide you through this. Sometimes we may get more than one pull requests. We typically accept the most quality contribution followed by the one that is made first. Happy contributing.
I would suggest we remove "those" and keep "the" so we are not repeating "those". I would also suggest using "attribute value"
The next paragraph reads pretty bad as well. It's just confusing to read. Maybe we can come up with something better. Edit: take two Or something like that?
I know the first line already mentioned "paragraph elements", so it is presumed that the second line is referring to that type of element. But as a newbie, when I read "to select the element with that attribute value", it seems any kind of element. Could we be more specific, and edit to "(...) to select the paragraph element (...)"?
It is selecting by so the element type doesn't matter. Saying what the element type is seems redundant to me. But if you or anyone else feel it would make it more clear, then I wouldn't be opposed to it.
Yes, you are right. Mentioning is what matters.

Related discussion:
text: # --description-- <del> You used the `inputVal` property as the `id` attribute for your paragraph elements. Now you can target those the paragraph elements with those `id`s. </del> <ins> You have set the `id` attribute for your paragraph elements to the `obj.inputVal` property. </ins> <del> Use the `.getElementById()` method to select the element with the `id` attribute with the value of the `inputVal` property of the current object. </del> <ins> Now, use `getElementById` to select the element with that attribute value, again using the `obj.inputVal` property. </ins> # --hints--
Code change:
Thanks for opening this issue. This looks like something that can be fixed by "first-time" code contributors to this repository. Here are the files that you should be looking at to work on a fix: File: Please make sure you read our , we prioritize contributors following the instructions in our guides. Join us in our or our if you need help contributing; our moderators will guide you through this. Sometimes we may get more than one pull request. We typically accept the most quality contribution followed by the one that is made first. Happy contributing.

Related discussion:
text: assert.match(calculateCalories.toString(), /document.querySelectorAll(s*('|")#breakfast input[s*type=numbers*]s*1s*)/); ``` <del> You should assign the result of your `odocument.querySelectorAll()` call to `breakfastNumberInputs`. </del> <ins> You should assign the result of your `document.querySelectorAll()` call to `breakfastNumberInputs`. </ins> ```js assert.match(calculateCalories.toString(), /breakfastNumberInputss*=s*document.querySelectorAll(s*('|")#breakfast input[s*type=numbers*]s*1s*)/);
Code change:
In step 10 of Luhn Algorithm, we teach campers how to create a substring using the operator. I think the explanation is a little short and could improved by: Adding comment next to each example, showing the result of each operation (we are currently communicating the value via the variable name, but I don't think it's clear enough) Mention that the starting index is inclusive, and the ending index is exclusive Additionally, I think we could: Change to to have the syntax more self-descriptive Mention that the default value of is 1 Perhaps the description should be changed to: python mystring = 'camperbot' mystring[0:6] == 'camper' # True my_string[0:6:3] == 'cp' # True `
Different project but same kind of example: I remember we decided not to use because of translations, but using them in the code example and then explain what they mean is fine :+1:
are we able to open this up for contribution?
Personally, I'd prefer to change the snippet into the following: I agree with all the rest.
I've updated the issue description with Dario's code snippet. This issue is open for contributions. We typically do not assign issues. Instead, we accept the most quality contribution followed by the one that is made first. Please make sure you read our as well as our , we prioritize contributors following the instructions in our guides. Join us in our or our if you need help contributing; our moderators will guide you through this. Happy contributing.
Whoops, sorry. I just noticed that Farhan is assigned to the issue. Are you working on this or could we mark this one up-for-grabs?
Yeah please mark this as up for grab

Related discussion:
text: # --description-- <del> You have accessed elements (characters) of a string before, using the index operator `[]`. You can also use the index operator to access a range of characters in a string with `string[x:y:h]`: </del> <ins> You have accessed elements (characters) of a string before, using the index operator `[]`. You can also use the index operator to access a range of characters in a string with `string[start:stop:step]`: </ins> ```python my_string = 'camperbot' <del> camper = my_string[0:6] cp = my_string[0:6:3] </del> <ins> my_string[0:6] == 'camper' # True my_string[0:6:3] == 'cp' # True </ins> ``` <del> Where `x` is the starting index, `y` is the ending index, and `h` is the step (the amount of characters to skip over). </del> <ins> Where `start` is the starting index (inclusive), `stop` is the ending index (exclusive), and `step` is the amount of characters to skip over. If not specified, `step` is default to 1. </ins> Create a variable named `card_number_reversed` and assign it the value of the first 4 characters of `card_number`.