jiebi/CodeConvo huggingface dataset:
Code change:
This part is still tripping people up Usually one piece or another gets lost: Maybe this is clearer Side note - I still don't love the fact that we say like a variable instead of like a string literal. I think that's confusing here. (I know, I know, the style guide hates "s) No response All No response
Agreed, it has the potential to be confusing. Can we reword to say: "If is the string and ..."
I'm still not clear on the reason why string literals in inline code blocks are forbidden by the style guide since it makes the instructions less clear in this case. do you have some context?
I don't remember the full reason, only that it was a decision made as part of our i18n efforts. We reference a discussion in but I can't for the life of me find where that discussion is.

Related discussion:
text: -   Your function must always return the entire `records` object. -   If `value` is an empty string, delete the given `prop` property from the album. -   If `prop` isn't `tracks` and `value` isn't an empty string, assign the `value` to that album's `prop`. <del> -   If `prop` is `tracks` and value isn't an empty string, add the `value` to the end of the album's `tracks` array. You need to create this array first if the album does not have a `tracks` property. </del> <ins> -   If `prop` is `tracks` and `value` isn't an empty string, you need to update the album's `tracks` array. First, if the album does not have a `tracks` property, assign it an empty array. Then add the `value` as the last item in the album's `tracks` array. </ins> **Note:** A copy of the `recordCollection` object is used for the tests. You should not directly modify the `recordCollection` object.
Code change:
In the new JS curriculum I found a couple of example where comments have been inside code blocks in the challenge description. These comments are not translatable on Crowdin: We should avoid commenting code blocks or make those comments accessible from Crowdin

Related discussion:
text: ```js switch (expression) { case 1: <del> /* this code will execute if the case matches the expression */ </del> <ins> console.log("1"); </ins> break; default: <del> /* the default will execute if none of the other cases match the expression */ </del> <ins> console.log("Not 1"); </ins> } ``` <ins> In this example, the first `case` checks if `expression === 1`, and if so logs `1` to the console. If no cases are true, it will log `Not 1` to the console. The `break` statement stops execution of the `switch` when the `case` is true. </ins> Add a `switch` statement and use `e.target.value` for the expression. # --hints--
Code change:
The setup instructions here suggest that I need MongoDB Community Server | 4.2.x. I've tried through docker and through macos homebrew instructions and can't get any 4.2 version. 4.4.x is available. Is that acceptable? If so, should I update this doc? Try to follow one of options for setting up MongoDB Community Server | 4.2.x Maybe 4.4.x is ok? No response
i searched the tags on docker and no 4.2 is available. this article suggests other ways but the result is always
i found this pr which suggests that 4.4 should work fine. so i guess i should update the docs? you agree?
or there are other docker images available of 4.2.x. But people trying to follow directions won't get to them if we point them to starting from here: because mongo only points to their docker images.
i also see that in there are docker instructions there. should i use that or is that for something different?
Hi thanks for the issue. We recommend asking around in our chat and forum when you are troubleshooting. The docs need to be updated to the version we are using right now. I have created a PR. It will be available soon at I hope this helps.
I see that the docs have been updated to mongo 5.0. But maybe should the docs point to as the best way to set up mongo? That's what I used and it seems like the easiest option.
Thanks for pointing that out. The directory is a newer API in the works. It will have a full docker-based setup that is more convenient than the current workflow. We will release cleaner and easy-to-follow docs when that happens. Meanwhile we will stick to the docs you see at:

Related discussion:
text: | --------------------------------------------------------------------------------------------- | ------- | ------------------------------------------------------------------------------------------- | | [Node.js](http://nodejs.org)                                                                  | `18.x`  | We use the "Active LTS" version, See [LTS Schedule](https://nodejs.org/en/about/releases/). | | [pnpm](https://pnpm.io/installation)                                                          | `8.x`   | -                                                                                            | <del> | [MongoDB Community Server](https://docs.mongodb.com/manual/administration/install-community/) | `4.2.x` | -                                                                                           | </del> <ins> | [MongoDB Community Server](https://docs.mongodb.com/manual/administration/install-community/) | `5.0.x` | -                                                                                           | </ins> > [!ATTENTION] > If you have a different version, please install the recommended version. We can only support installation issues for recommended versions. See [troubleshooting](#troubleshooting) for details.
Code change:
The formula given for converting from C to F is given as a) But this can also be written as b) These two formulas are producing different results. Using for (which is used in the tests) we get: a) b) If formula is used then the tests for this challenge will fail due to the floating point rounding issue. The tests should pass regardless of where C is placed in the formula. Either we need to do a little rounding in the tests or replace with a different number in the tests that won't produce different results depending on the placement of C in the formula. No response Device: Desktop OS: Linux/Fedora 36 Browser: Firefox Version: 112 No response

Related discussion:
text: `Thermostat` should be a `class` with a defined `constructor` method. ```js <del> assert( typeof Thermostat === 'function' && typeof Thermostat.constructor === 'function' ); </del> <ins> assert.isFunction(Thermostat); assert.isFunction(Thermostat?.constructor); </ins> ``` <del> `class` keyword should be used. </del> <ins> The `class` keyword should be used. </ins> ```js <del> assert(code.match(/class/g)); </del> <ins> assert.match(code, /class/); </ins> ``` `Thermostat` should be able to be instantiated. ```js <del> assert( (() => { const t = new Thermostat(122); return typeof t === 'object'; })() ); </del> <ins> const _t = new Thermostat(122); assert.isObject(_t); </ins> ``` When instantiated with a Fahrenheit value, `Thermostat` should set the correct `temperature`. ```js <del> assert( (() => { const t = new Thermostat(122); return t.temperature === 50; })() ); </del> <ins> const _t = new Thermostat(122); assert.strictEqual(_t?.temperature, 50); </ins> ``` A `getter` should be defined. ```js <del> assert( (() => { const desc = Object.getOwnPropertyDescriptor( Thermostat.prototype, 'temperature' ); return !!desc && typeof desc.get === 'function'; })() ); </del> <ins> const _desc = Object.getOwnPropertyDescriptor(Thermostat.prototype, 'temperature'); assert.isFunction(_desc?.get); </ins> ``` A `setter` should be defined. ```js <del> assert( (() => { const desc = Object.getOwnPropertyDescriptor( Thermostat.prototype, 'temperature' ); return !!desc && typeof desc.set === 'function'; })() ); </del> <ins> const _desc = Object.getOwnPropertyDescriptor(Thermostat.prototype, 'temperature'); assert.isFunction(_desc?.set); </ins> ``` Calling the `setter` with a Celsius value should set the `temperature`. ```js <del> assert( (() => { const t = new Thermostat(32); t.temperature = 26; const u = new Thermostat(32); u.temperature = 50; return t.temperature === 26 && u.temperature === 50; })() ); </del> <ins> const _t = new Thermostat(32); _t.temperature = 26; const _u = new Thermostat(32); _u.temperature = 50; assert.approximately(_t.temperature, 26, 0.1); assert.approximately(_u.temperature, 50, 0.1); </ins> ``` # --seed--
Code change:
As i love free code ; I would like to inform the problem and stay as minimal as i can. staight on problem. in : Responsive Web Design Learn More About CSS Pseudo Selectors By Building A Balance SheetQUALITY CONTROL DEGRADING CODE ROBUSTNESS. 47: <tr class="data"48: <thSavings <span class="description"Funds set aside for emergencies.</span</th49: <td650</td51: <td 500</td<td728</td<---------- answered passed without this line of code. </tr! ! Device: [Asus f17 gaming] OS: [ Windows 11] Browser: [Chrome] thanks for such a go tutorial platform!
Hi thanks for the report. Could you share your full code here so that we can dig into what's going wrong?
Edit: I backticks for clarity.
Thanks for sharing that but the code you shared passes the tests. That's expected, since it has in the correct place. If I remove then it fails, as it should. From your screenshot, it looked like you'd entered invalid code (i.e. without ) and managed to pass the tests. However, I've been unable to replicate this behaviour. If you have an example of invalid code that passes, could you share it here?
first screen shot mentions step 15, the second one shows 16. Isn't the issue in the step 16?
but it is there in step 16 code
yes in hurry i was not able to give accurate problem details. IT IS IN STEP 16 THANKS BRO
Silly me newbie.
Okay, that makes sense, thanks for clarifying. To fix this, there needs to be one more test that asserts that is set on the third element.
sir did u get what i want to say? as if not possible is it allowed to express it on video call? if okay so? like on ms teams.
Hey we understand the issue and have a pull request open to address it. Thank you for reporting. We will get the fix in as soon as we can

Related discussion:
text: assert(document.querySelector('tbody')?.querySelectorAll('tr')?.[3]?.querySelectorAll('td')?.[2]?.textContent === '$809'); ``` <ins> Your third `td` element should have the `class` attribute set to `current`. ```js assert(document.querySelector('tbody')?.querySelectorAll('tr')?.[3]?.querySelectorAll('td')?.[2]?.classList?.contains('current')); ``` </ins> # --seed-- ## --seed-contents--
Code change:
The English curriculum was as an upcoming change. A new "Fill in the blank" style challenge was created for it. When you go to one of these challenges and try to type an , it doesn't show up (sometimes). If you type it really quickly after a different character, it does show up. I'm quite sure the hotkeys are interfering - if you turn them off, the problem goes away. The is the one that we noticed, but there could be other characters that have this issue as well. Fill in the blank style challenges the codebase with the upcoming changes shown to one of the fill in the blank challenges: http://localhost:8000/learn/a2-english-for-developers/learn-greetings-in-your-first-day-at-the-office/right to type an in the blank, it doesn't work of keyboard shortcuts in settings, it does work All keys to work all the time. Related discussion:

Related discussion:
text: <Spacer size='medium' /> <h2>{t('learn.fill-in-the-blank')}</h2> <Spacer size='small' /> <del> <ObserveKeys> </del> <ins> {/* what we want to observe is ctrl/cmd + enter, but ObserveKeys is buggy and throws an error if it encounters a key combination, so we have to pass in the individual keys to observe */} <ObserveKeys only={['ctrl', 'cmd', 'enter']}> </ins> <div> <p> {splitSentence.map((s, i) => {
Code change:
The following valid code does not pass for of calorie counter project. The issue is with this line here If you remove the semicolon then the test will pass We will need to fix the test to make semicolons optional so the camper can pass with or without semicolons. here is the file you need to fix Please read through the docs first. And we don't assign issues. If you are interested in working on this, work on the fix and create a PR. Happy coding! This code does not pass even though it is valid Semicolons should be optional and the code above should pass 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

Related discussion:
text: Your `for` loop should `push` `strArray[i]` to `cleanStrArray`. ```js <del> assert.match(code, /fors*(s*(let|var)s+is*=s*0s*;s*is*<s*strArray.lengths*;s*is*++s*)s*{s*ifs*(![("|')+2s*,s*("|')-3s*,s*("|')s4s*].includes(s*strArray[i]s*))s*{s*cleanStrArray.push(s*strArray[i]s*)s*}/); </del> <ins> assert.match(code, /fors*(s*(let|var)s+is*=s*0s*;s*is*<s*strArray.lengths*;s*is*++s*)s*{s*ifs*(![("|')+2s*,s*("|')-3s*,s*("|')s4s*].includes(s*strArray[i]s*))s*{s*cleanStrArray.push(s*strArray[i]s*)s*;?s*}/); </ins> ``` # --seed--
Code change:
The test is failing if there isn't a space between and . This fails: Test should pass without a space between and . No response N/A No response

Related discussion:
text: You should add an `else` block to the existing `if` block. ```js <del> assert.match(code, /ifs*(userData?.currentSongs*===s*nulls*||s*userData?.currentSong.ids*!==s*song.id)s*{s*audio.currentTimes*=s*0;?s*}s*elses{/) </del> <ins> assert.match(code, /ifs*(userData?.currentSongs*===s*nulls*||s*userData?.currentSong.ids*!==s*song.id)s*{s*audio.currentTimes*=s*0;?s*}s*elses*{/) </ins> ``` You should set `audio.currentTime` to `userData.songCurrentTime` inside the `else` block of your `if` statement. ```js <del> assert.match(code, /ifs*(userData?.currentSongs*===s*nulls*||s*userData?.currentSong.ids*!==s*song.id)s*{s*audio.currentTimes*=s*0;?s*}s*elses{s*audio.currentTimes*=s*userData.songCurrentTime;?s*}/) </del> <ins> assert.match(code, /ifs*(userData?.currentSongs*===s*nulls*||s*userData?.currentSong.ids*!==s*song.id)s*{s*audio.currentTimes*=s*0;?s*}s*elses*{s*audio.currentTimes*=s*userData.songCurrentTime;?s*}/) </ins> ``` # --seed--
Code change:
The fill-in-the-blank sentence doesn't match what the character says. The written sentence is: The character says: Note the "the", instead of "our". No response Device: all OS: all Browser: all Version: all No response
Thank you for reporting the issue. Note for maintainers: Step 54 () uses "our" instead of "the" in the quiz. I think it's fine as-is since there isn't associated audio/caption text, but I'd vote updating it, too, for consistency. This issue is open to first-time code contributors to this repository. Requirements: In Change the text in the tag from to In Change the text in the tag from to Change the fill-in-the-blank text from to Change the backtick placement in the description text from ` In Change the text of the answers from to 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've created a PR addressing these changes.

Related discussion:
text: dashedName: task-50 --- <del> <!-- (audio) Tom: Those are great traits for someone in her position, I think. What does she do as our team lead? --> </del> <ins> <!-- (audio) Tom: Those are great traits for someone in her position, I think. What does she do as the team lead? --> </ins> # --description--
Code change:
The fill-in-the-blank sentence doesn't match what the character says. The written sentence is: The character says: Note the "the", instead of "our". No response Device: all OS: all Browser: all Version: all No response
Thank you for reporting the issue. Note for maintainers: Step 54 () uses "our" instead of "the" in the quiz. I think it's fine as-is since there isn't associated audio/caption text, but I'd vote updating it, too, for consistency. This issue is open to first-time code contributors to this repository. Requirements: In Change the text in the tag from to In Change the text in the tag from to Change the fill-in-the-blank text from to Change the backtick placement in the description text from ` In Change the text of the answers from to 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've created a PR addressing these changes.

Related discussion:
text: ## --answers-- <del> `What do she does as our team lead?` </del> <ins> `What do she does as the team lead?` </ins> ### --feedback--
Code change:
The fill-in-the-blank sentence doesn't match what the character says. The written sentence is: The character says: Note the "the", instead of "our". No response Device: all OS: all Browser: all Version: all No response
Thank you for reporting the issue. Note for maintainers: Step 54 () uses "our" instead of "the" in the quiz. I think it's fine as-is since there isn't associated audio/caption text, but I'd vote updating it, too, for consistency. This issue is open to first-time code contributors to this repository. Requirements: In Change the text in the tag from to In Change the text in the tag from to Change the fill-in-the-blank text from to Change the backtick placement in the description text from ` In Change the text of the answers from to 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've created a PR addressing these changes.

Related discussion:
text: --- <del> `What does he does as our team lead?` </del> <ins> `What does he does as the team lead?` </ins> ### --feedback--
Code change:
The fill-in-the-blank sentence doesn't match what the character says. The written sentence is: The character says: Note the "the", instead of "our". No response Device: all OS: all Browser: all Version: all No response
Thank you for reporting the issue. Note for maintainers: Step 54 () uses "our" instead of "the" in the quiz. I think it's fine as-is since there isn't associated audio/caption text, but I'd vote updating it, too, for consistency. This issue is open to first-time code contributors to this repository. Requirements: In Change the text in the tag from to In Change the text in the tag from to Change the fill-in-the-blank text from to Change the backtick placement in the description text from ` In Change the text of the answers from to 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've created a PR addressing these changes.

Related discussion:
text: --- <del> `What do he does as our team lead?` </del> <ins> `What do he does as the team lead?` </ins> ### --feedback--
Code change:
The fill-in-the-blank sentence doesn't match what the character says. The written sentence is: The character says: Note the "the", instead of "our". No response Device: all OS: all Browser: all Version: all No response
Thank you for reporting the issue. Note for maintainers: Step 54 () uses "our" instead of "the" in the quiz. I think it's fine as-is since there isn't associated audio/caption text, but I'd vote updating it, too, for consistency. This issue is open to first-time code contributors to this repository. Requirements: In Change the text in the tag from to In Change the text in the tag from to Change the fill-in-the-blank text from to Change the backtick placement in the description text from ` In Change the text of the answers from to 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've created a PR addressing these changes.

Related discussion:
text: --- <del> `What does she do as our team lead?` </del> <ins> `What does she do as the team lead?` </ins> ## --video-solution--
Code change:
The fill-in-the-blank sentence doesn't match what the character says. The written sentence is: The character says: Note the "the", instead of "our". No response Device: all OS: all Browser: all Version: all No response
Thank you for reporting the issue. Note for maintainers: Step 54 () uses "our" instead of "the" in the quiz. I think it's fine as-is since there isn't associated audio/caption text, but I'd vote updating it, too, for consistency. This issue is open to first-time code contributors to this repository. Requirements: In Change the text in the tag from to In Change the text in the tag from to Change the fill-in-the-blank text from to Change the backtick placement in the description text from ` In Change the text of the answers from to 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've created a PR addressing these changes.

Related discussion:
text: dashedName: task-55 --- <del> <!-- (audio) Tom: Those are great traits for someone in her position, I think. What does she do as our team lead? --> </del> <ins> <!-- (audio) Tom: Those are great traits for someone in her position, I think. What does she do as the team lead? --> </ins> # --description-- <del> Now you will practice listening for the correct form of `do.` Listen carefully to how Tom asks about Maria's job as a team lead. </del> <ins> Now you will practice listening for the correct form of `do`. Listen carefully to how Tom asks about Maria's job as a team lead. </ins> # --instructions--
Code change:
The fill-in-the-blank sentence doesn't match what the character says. The written sentence is: The character says: Note the "the", instead of "our". No response Device: all OS: all Browser: all Version: all No response
Thank you for reporting the issue. Note for maintainers: Step 54 () uses "our" instead of "the" in the quiz. I think it's fine as-is since there isn't associated audio/caption text, but I'd vote updating it, too, for consistency. This issue is open to first-time code contributors to this repository. Requirements: In Change the text in the tag from to In Change the text in the tag from to Change the fill-in-the-blank text from to Change the backtick placement in the description text from ` In Change the text of the answers from to 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've created a PR addressing these changes.

Related discussion:
text: ## --sentence-- <del> `Those are great traits for someone in her position, I think. What _ she _ as our team lead?` </del> <ins> `Those are great traits for someone in her position, I think. What _ she _ as the team lead?` </ins> ## --blanks--