jiebi/CodeConvo huggingface dataset:
Code change:
Step 22 of build a Todo app can use a better a hint or a short reminder on destructuring if one can be into the instructions. If we forget to put the curly braces around the parameter names to destructure them, the hint only says: You should destructure id, title, date, description as the parameters of the callback of your forEach(). A better hint would be I think parameter destructuring was only covered once before this project. (though I may recall that incorrectly) here: Either a better hint or a reminder of parameter destructuring in the instructions would be helpful. No response N/A No response
Hello. I would like to work on this issue. Please assign this to me.
Let's go with this text.

Related discussion:
text: assert.match(code, /taskData.forEach(s*/) ``` <del> You should destructure `id`, `title`, `date`, `description` as the parameters of the callback of your `forEach()`. </del> <ins> You should use `{id, title, date, description}` as the destructured parameters of the `forEach()` callback function. </ins> ```js assert.match(code, /taskData.forEach(s*(s*{s*(?:ids*,s*titles*,s*dates*,s*description)|(?:titles*,s*ids*,s*dates*,s*description)|(?:dates*,s*ids*,s*titles*,s*description)|(?:ids*,s*dates*,s*titles*,s*description)|(?:titles*,s*dates*,s*ids*,s*description)|(?:dates*,s*titles*,s*ids*,s*description)|(?:dates*,s*titles*,s*descriptions*,s*id)|(?:titles*,s*dates*,s*descriptions*,s*id)|(?:descriptions*,s*dates*,s*titles*,s*id)|(?:dates*,s*descriptions*,s*titles*,s*id)|(?:titles*,s*descriptions*,s*dates*,s*id)|(?:descriptions*,s*titles*,s*dates*,s*id)|(?:descriptions*,s*ids*,s*dates*,s*title)|(?:ids*,s*descriptions*,s*dates*,s*title)|(?:dates*,s*descriptions*,s*ids*,s*title)|(?:descriptions*,s*dates*,s*ids*,s*title)|(?:ids*,s*dates*,s*descriptions*,s*title)|(?:dates*,s*ids*,s*descriptions*,s*title)|(?:titles*,s*ids*,s*descriptions*,s*date)|(?:ids*,s*titles*,s*descriptions*,s*date)|(?:descriptions*,s*titles*,s*ids*,s*date)|(?:titles*,s*descriptions*,s*ids*,s*date)|(?:ids*,s*descriptions*,s*titles*,s*date)|(?:descriptions*,s*ids*,s*titles*,s*date)s*}/)
Code change:
The instructions for step 103 of the spreadsheet project request that a range be created from an array. The say: My solution (listed below) tries to do that by passing in the first number found in nums and the last number found in nums as arguments to the range function. This solution is not accepted and the hint says: On reading this initially I have no clue what this hint is attempting to say. Does it want the range to span from the first number of the array to the second number only? Does it want the range property to define a function that takes two arguments? I'm really not sure what it wants. After some investigations though it seems that this step wanted the range to be between the first and second numbers of the array. I think this confusion could have been easily avoided if the instructions just said that. Change the instructions to say: numsnums No response N/A No response
One of the tests is checking , so it's clearly intended to use first and second number.

Related discussion:
text: # --description-- <del> Add a `range` property which generates a range from `nums`. Remember that you have a `range` function you can reuse here. </del> <ins> Add a `range` property which generates a range from the first number in `nums` to the second number in `nums`. Remember that you have a `range` function you can reuse here. </ins> # --hints--
Code change:
The code should not be centered in the preview modal. or to on button for certification project with code ! ! 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_

Related discussion:
text: </Table> )} {id && ( <del> <Modal onClose={closeSolution} open={solutionOpen}> </del> <ins> <Modal onClose={closeSolution} open={solutionOpen} size='large'> </ins> <Modal.Header showCloseButton={true} closeButtonClassNames='close'> {`${username}'s Solution to ${ idToNameMap.get(id)?.challengeTitle ?? '' }`} </Modal.Header> <del> <Modal.Body> </del> <ins> <Modal.Body alignment='left'> </ins> <SolutionViewer challengeFiles={challengeData.challengeFiles} solution={challengeData.solution ?? ''}
Code change:
typo error on the console warning: If a large straight is rolled, your checkForStraights function should enable the fifth radio button, set the value to 40, and update the displayed text to , score = 0. i recreated the last step to get to the warnings. it's supposed to say :If a large straight is rolled, your checkForStraights function should enable the fifth radio button, set the value to 40, and update the displayed text to , score = 40. ---- not 0. <img width="442" alt="Screenshot 2024-08-25 at 8 08 39 PM" src=""- Device: Mac OS: Ventura 13.6.9 Browser: Brave, Safari Version: Brave - , Safari - Version 17.6 (.3.11.11.7, ) 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.

Related discussion:
text: assert.strictEqual(scoreSpans[3].innerText, ", score = 30"); ``` <del> If a large straight is rolled, your `checkForStraights` function should enable the fifth radio button, set the value to `40`, and update the displayed text to `, score = 0`. </del> <ins> If a large straight is rolled, your `checkForStraights` function should enable the fifth radio button, set the value to `40`, and update the displayed text to `, score = 40`. </ins> ```js resetRadioOptions();
Code change:
The challenge or lesson asks the user to provide answer by looking at or by executing the sample code provided in front of him/her but the sample doesn't provide any output in console. When a user try that piece of code he/she should see an output in console. 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
Can I work on this and just add the myFunction(); at the end?
hi Thanks for your interest in contributing. This issue has not been triaged yet, and a solution has not been agreed upon. Once the issue is open for contribution, you are welcome to create a pull request to reflect the issue consensus. Please also note that we typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue.
Ok, yes, it makes sense to add a function call in the code snippet. I'm opening this up for contributors to work on. This is the piece of code that needs to be changed: 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.
Hi I'd like to work on this issue and submit a pull request to address it. This will be my first issue. I just wanted to know if the video-solution is correct on that page. Just seems to say '2'. Is that intentional or is it missing a zero. Either way I'll add the function call in the code snippet and send a merge req right away. Thanks!
Hey We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labeled with or are open for contributions. Please make sure you read . We prioritize contributors following the instructions in our guide. Join us in or if you need help contributing - our community will be happy to assist you.

Related discussion:
text: let x = 20; console.log(x); } <ins> myFunction(); </ins> ``` ## --answers--
Code change:
I suggest changing the phrase 'template interpolation' to 'template literals,' because are correctly referred to as template literals, which support string interpolation. ! 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_
it should be "string interpolation". Template literals are already mentioned a few words before, it would be extremely redundant. I'm opening this issue for first timers to make the change "template interpolation" -"string interpolation". This change needs to be done twice in this 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.
Sorry for the misunderstanding. I need to change the text from 'template interpolation' -'string interpolation'
Hi, I guess has raised his PR with the needed change. I haven't contributed before to any open source project. I thought this was an easy one to do to get started. Please tell any easy code or non-code contributions to be made. Happy to help.
I am able to make the required change and happy to start open source contribution with hacktoberfest . Kindly assign this issue to me.
We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labeled with or are open for contributions. Please make sure you read . We prioritize contributors following the instructions in our guide. Join us in or if you need help contributing - our community will be happy to assist you.
This pull request resolves the issue of incorrect terminology in the JavaScript curriculum where "template interpolation" was used. The correct term is "string interpolation," as template literals support string interpolation. I have made the following changes: Replaced "template interpolation" with "string interpolation" in the two locations within the file. Fix :
After reading the contributing guidelines, I now understand the mistake I made in my PR.

Related discussion:
text: # --description-- <del> Inside the template literal, create a `div` element with the `id` set to the `index` from the `.forEach()` array method. Remember to use template interpolation to do this. </del> <ins> Inside the template literal, create a `div` element with the `id` set to the `index` from the `.forEach()` array method. Remember to use string interpolation to do this. </ins> Also, add a `class` of `"user-card"` to the `div`.
Code change:
I suggest changing the phrase 'template interpolation' to 'template literals,' because are correctly referred to as template literals, which support string interpolation. ! 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_
it should be "string interpolation". Template literals are already mentioned a few words before, it would be extremely redundant. I'm opening this issue for first timers to make the change "template interpolation" -"string interpolation". This change needs to be done twice in this 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.
Sorry for the misunderstanding. I need to change the text from 'template interpolation' -'string interpolation'
Hi, I guess has raised his PR with the needed change. I haven't contributed before to any open source project. I thought this was an easy one to do to get started. Please tell any easy code or non-code contributions to be made. Happy to help.
I am able to make the required change and happy to start open source contribution with hacktoberfest . Kindly assign this issue to me.
We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labeled with or are open for contributions. Please make sure you read . We prioritize contributors following the instructions in our guide. Join us in or if you need help contributing - our community will be happy to assist you.
This pull request resolves the issue of incorrect terminology in the JavaScript curriculum where "template interpolation" was used. The correct term is "string interpolation," as template literals support string interpolation. I have made the following changes: Replaced "template interpolation" with "string interpolation" in the two locations within the file. Fix :
After reading the contributing guidelines, I now understand the mistake I made in my PR.

Related discussion:
text: assert.include(document.querySelector('#author-container > div')?.className, 'user-card'); ``` <del> Your `div` element should have the `id` of `index`. Remember to use template interpolation to set the `id`. </del> <ins> Your `div` element should have the `id` of `index`. Remember to use string interpolation to set the `id`. </ins> ```js displayAuthors([{author: "Naomi"}]);