jiebi/CodeConvo huggingface dataset:
Change:
const ctx: Worker & typeof globalThis = self as unknown as Worker & typeof globalThis; <del> let pyodide: PyodideInterface; </del> <ins> let pyodide: PyodideInterface | null = null; </ins> interface PythonRunEvent extends MessageEvent { data: {

Discussion:
Pyodide can crash when adding or method to the class, while NOT trying to write invalid code. I'm mentioning intent, because it can be encountered when attempting to write correct code. ing instance of the class. or method to the class. On each key pressed, code will execute, including the newly written method. So when we are writing the method: Then starting writing inside of the brackets: . Code will be execute also when it's . That's when it will crash hard: Further changes to code will bring up in browser's console: With example challenge to the at the bottom of code. to class definition, add boilerplate: writing between brackets: on the right crashes at the point when it's . Ideally omitting the crashing part. After the crash window needs to be reloaded to make pyodide on the right work again. Tests still appear to work correctly. Recursive function going into maximum call stack size exceeded does not crash pyodide. I've tested example class from above on , it does not crash pyodide.
Change:
You have either deleted the `html` element or it is missing an opening tag or closing tag. ```js <del> assert(code.match(/<html.*?>/) && code.match(/</html>/)); </del> <ins> assert.match(code, /<html.*?>/); assert.match(code, /</html>/); </ins> ``` Your `html` element should have a `lang` attribute with the value `en`. You may have omitted the attribute/value, or have a typo. ```js const extraSpacesRemoved = code.replace(/s+/g, " "); <del> assert(extraSpacesRemoved.match(/<htmls+langs*=s*("?|'?)en1s*>/)); </del> <ins> assert.match(extraSpacesRemoved, /<htmls+langs*=s*("?|'?)en1s*>/); </ins> ``` # --seed--

Discussion:
We recently updated the guidelines for writing assertions here For consistency and matching what is in the docs, we should slowly start to update the tests for the workshops and labs in the new frontend cert. For this issue, just focus on updating the tests for steps 36 - 40 of the cat photo app workshop. We don't want to have a large PR that updates a whole bunch of tests in a bunch of steps and makes it harder to review. Here is the link to the workshop: The tests should be updated, where appropriate, to use specific methods like , , , etc instead of just . Here is the complete API reference Just focus on the new frontend cert and don't worry about updating the RWD cat photo app since that will become legacy anyway.
Change:
"  ", "  ", "</style>", <del> "<a href="https://atjonathan.github.io/catphotoapp" target="_blank">CatPhotoApp</a>" </del> <ins> "<a href="http://freecatphotoapp.com/" target="_blank">CatPhotoApp</a>" </ins> ], "tests": [ "assert($('a').css('color') == 'rgb(0, 0, 0)', 'message: The <code>color</code> of the anchor tag text should be black.')"

Discussion:
We need to update the links in the challenges from to Changes are to be made here: and here:
I can take it! :)
please go ahead! Good luck. Here are the .
Donezo :). Thanks!
Change:
style={characterFeatureStyles} className='character-feature' src={base} <del> alt='Character Body' </del> <ins> alt='' </ins> /> <img style={characterFeatureStyles} className='character-feature' src={brows} <del> alt='Character Brows' </del> <ins> alt='' </ins> /> <img style={characterFeatureStyles} className='character-feature' src={eyesAreOpen ? eyesOpen : eyesClosed} <del> alt='Character Eyes' </del> <ins> alt='' </ins> /> <img style={characterFeatureStyles} className='character-feature' src={mouthIsOpen ? mouthOpen : mouthClosed} <del> alt='Character Mouth' </del> <ins> alt='' </ins> /> {glasses && ( <img style={characterFeatureStyles} className='character-feature' src={glasses} <del> alt='Character Glasses' </del> <ins> alt='' </ins> /> )} </div>

Discussion:
Originally posted by in We should hide the character images with - we can probably hide all the images, except for maybe the base character.
I believe leaving blank should suffice: I haven't tested locally and only skimmed through the code. Correct me if I'm wrong, I think we are communicating which character is speaking only visually using these images. So hiding the images is one thing, but providing an alternative way to convey the information is another thing.
Yes, definitely, these images should have empty alt text!
Alt text emptied
Change:
# --hints-- <del> Your `.cat-whisker-right-top` selector should have a `transform` property set to `rotate(-1deg)`. </del> <ins> Your `.cat-whisker-right-top` selector should have a `transform` property set to `rotate(-10deg)`. </ins> ```js assert(new __helpers.CSSHelp(document).getStyle('.cat-whisker-right-top')?.transform === 'rotate(-10deg)')

Discussion:
instructions state -10px failed test done incorrectly. error states that it must be -1degerror instruction bug proven. please be grateful free testing again to your company. I would like to one day do corrections to updated things like this on github, step 78 new responsive web design did not pass with clear instructions. error says -1deg ! 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
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: List of files: The test text should be updated 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.
Open Source is based upon free contributions. We're always grateful for efforts that move the codebase forward.
Change:
If the OP (Original Poster) replies within a freeCodeCamp curriculum help thread with their final solution, blur it and use the **Blurred Spoiler Solution** canned reply. <del> If a user creates a thread asking for feedback on a solution, move the thread to the feedback subforum and blur the solution, as necessary. If the user is only posting the solution to show if off, then unlist the thread and use the **Solutions Thread** canned reply. </del> <ins> If a user creates a thread asking for feedback on a solution, move the thread to the feedback subforum and blur the solution, as necessary. If the user is only posting the solution to show it off, then unlist the thread and use the **Solutions Thread** canned reply. </ins> ### Underage Users

Discussion:
In the document, there is a sentence saying this: But this section is talking about dealing with solutions , it should be changed to this: here is the file If you would like to fix this issue, 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.
Hi there! I'm definitely a first-timer and would like to fix this. You can assign this issue to me.
Hi feel free to open a pull request after you have read our contribution guide
Hey there! I would like to fix this issue,you can assign this to me.
Hey feel free to open a pull request after you have read our contribution guide
Change:
showPoints: PropTypes.bool, showPortfolio: PropTypes.bool, showTimeLine: PropTypes.bool, <del> updateMyProfileUI: PropTypes.func.isRequired </del> <ins> updateMyProfileUI: PropTypes.func.isRequired, user: PropTypes.object </ins> }; function PrivacySettings(props) {

Discussion:
Once we've built the JSON we want to output, I believe we can just use Pug like this for the button to initiate the file download:
Change:
"dashedName": "build-a-palindrome-checker-project", "usesMultifileEditor": true, "helpCategory": "JavaScript", <del> "order": 8, </del> <ins> "order": 7, </ins> "superBlock": "javascript-algorithms-and-data-structures-v8", "challengeOrder": [ {

Discussion:
I was going through the new JS Beta curriculum and when I got to the date formatter project I realized that it probably doesn't really help add any specific knowledge that would be useful in completing the palindrome project. I think the date formatter project being in its current position may even be a hindrance because I think it is better to get people to work on the skills they have learned sooner rather than later. My suggestion is to move the palindrome project ahead of the date formatter project for this reason. N/A N/A No response N/A No response
please assign this to me
issues are not generally assigned, and this issue has not been marked as ready for contributions.
okiee thank you
I agree. That seems like the way to go. A PR which resolves this issue, then, will swap the properties in the meta files for these two projects.
Change:
```yml tests: <del> - text: declare the <code>--penguin-belly</code> variable in the <code>:root</code> and assign it to <code>pink</code>. testString: assert(code.match(/:roots*?{[sS]*--penguin-bellys*?:s*?pinks*?;[sS]*}/gi), 'declare the <code>--penguin-belly</code> variable in the <code>:root</code> and assign it to <code>pink</code>.'); </del> <ins> - text: Declare the <code>--penguin-belly</code> variable in the <code>:root</code> and assign it to <code>pink</code>. testString: assert(code.match(/:roots*?{[sS]*--penguin-bellys*?:s*?pinks*?;[sS]*}/gi), 'Declare the <code>--penguin-belly</code> variable in the <code>:root</code> and assign it to <code>pink</code>.'); </ins> ```

Discussion:
<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --Describe your problem and how to reproduce it: Challenge test text typically capitalized, but the one for Cascading CSS Variables is not. Add a Link to the page with the problem: Live: Suggested Change: Tell us about your browser and operating system: Browser Name: Chrome Browser Version: Chrome Operating System: Chrome OS
Hello can I work on this issue? I'd like to start contributing. kind regards!
Go for it! Thanks for giving us a hand! You will need to follow the to set up freeCodeCamp locally. Also, we are hanging out in . You can ask for help there, or just come say hi :)
Thanks, yeah sure, see you at Glitter's Contributors chatroom
is it fixed ? i would like to contribute
there has been a pull request already submitted Thanks for your interest! Please take a look at the other issues. Happy coding!
Close the issue?
Change:
**Examples** ```js <del> 4   <= 5 '7' <= 7 5   <= 5 3   <= 2 '8' <= 4 </del> <ins> 4   <= 5 // true '7' <= 7 // true 5   <= 5 // true 3   <= 2 // false '8' <= 4 // false </ins> ``` <del> In order, these expressions would evaluate to `true`, `true`, `true`, `false`, and `false`. </del> # --instructions-- Add the less than or equal to operator to the indicated lines so that the return statements make sense.

Discussion:
The student has to go all the way down up to the explanation "In order, these expressions would evaluate to true, false, true, and true." to realize what the result of the operations is. My suggestion is that the second section of the code should look like this: 1 == 1 // true 1 == 2 // false 1 == '1' // true "3" == 3 // true instead of the screenshot attached. <!-- Add a link to the coding challenge with the problem. -- <!-- If applicable, add screenshots to help explain your problem. you can drag and drop, png, jpg, gif, etc. in this box. --!
I would be fine with this change. Any interest in making a PR for it
, Can I? I don't know how to do it and I have never done it in my life, but besides that, I would like to do it by myself using some guide you could give me. I have never used GitHub in the past. And this will help me to contribute even more to this awesome project.
We have some . It may look a little intimidating, but if you follow the instructions it's not bad. The file where this needs to be fixed is Technically, you can edit the file on github and make a PR from there - which can work for some simple fixes, but you can't see and test your changes. I think we would accept those if you want to do that, but I would recommend setting the project up locally at some point if you want to contribute regularly. To be clear, we usually like to get second opinion before making changes like this - since we don't have that, I can't guarantee a PR would be merged - but I think others would be fine with the change so if you want to make a PR, go ahead.
This change seems good. For context, we got rid of a lot of comments since we'd need to translate them. However, these are keywords and don't need translation.
shall I take this up?
Could you clarify if this is actually what we wanted, because this is exactly what we removed 12 months ago in:
Yes, this is okay because these instances specifically are only keywords that don't get translated anyway.
Change:
Use the `setAttribute` method on the `playButton` element to set an attribute named `"aria-label"`. Using a ternary, set the attribute value to `Play ${song.title}` or `"Play"` if `song?.title` is not available. <del> Don't forget you need template interpolation here, so you need to use backticks. </del> <ins> Don't forget you need string interpolation here, so you need to use backticks. </ins> # --hints--

Discussion:
When talking about template literals sometimes we use "template interpolation", I have not been able to find good documentation on this wording. It should be changed to "string interpolation" or "template literal" as needed. The changes needed are as follow: template interpolation -string interpolation use template interpolation -use string interpolation 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.
Change:
```js assert.strictEqual( (function () { <del> bob.setFullName('Haskell Curry'); return bob.getFullName(); </del> <ins> _test_bob.setFullName('Haskell Curry'); return _test_bob.getFullName(); </ins> })(), 'Haskell Curry' );

Discussion:
A user can fail with correct code because the tests secretly rely upon global variables accessible to the user. This code fails because the user has the correct solution but deleted the instantiation of the object. Tests should instantiate any data they need. We should update the seed code to We should then add and modify the test suite to use instead of . No response All No response
Hello, Just to clarify, did you want to add in the user code block? Because I noticed when I attempted to reproduce the bug you encountered that two test cases still passed, despite not being initialized in the user's code. I checked the lesson's markdown file and found out that those two test cases did have their own initialized. I was thinking of adding to all the test cases instead. For reference, here's the code for one of the test cases that still passed, despite the bug.
Those tests initiate their own values so they are fine. I'm talking about the tests that fail if users modify their seed code to not have the object initialization. The new variable would go into a new --after-user-code-- part; that's not currently present. Just adding this new variable inside of the seed creates the exact same problem. Note - this issue isn't marked as ready for contributions yet. Maybe we should review if the tests logic too to see if they are independent
Opening this up for contribution.
Hi assign this issue for branch bugfix/. I have resolved it, need to create PR for same
We don't assign issues. The first PR that fully covers the issue will be accepted.
Hi ok !! I have raised PR . thanks for reply.
Change:
console.log(element.textContent); // Output: This is some text content ``` <ins> You can use a ternary operator to conditionally set the text content value. Here is an example of assigning the result of a ternary operator to a variable: ```js const example = condition ? "I'm true" : "I'm false"; ``` </ins> Use a ternary operator to check if `currentTitle` evaluates to a truthy value. If it does, set `playingSong.textContent` to `currentTitle`. Otherwise, set it to an empty string. Then below that, use a `ternary` operator to check if `currentArtist` is truthy. If so, set `songArtist.textContent` to `currentArtist`. Otherwise, set it to empty string.

Discussion:
in Step 70 of the music player project (js beta) the step requests a ternary statement that is assigned to a variable. People on the forum have a fair amount of struggles with this which is expected given that this may be the first time they are putting these two things together (ternary and assignment). But, now that I've personally moved on to the spam filter project, I noted a curious thing. Step 5 of the spam filter project has a handy-dandy example of exactly the type of thing needed back in the music player project: Here is an example of assigning the result of a ternary operator to an element's text content: Example Code So this issue is a request to consider adding a similar example into step 70. A quick search for this step in the forum will show many examples of people struggling with this code's construction as further evidence of need. I realise that this may make the text too long for step 70 though. If the example is , please modify it to include the 'else' section too. And maybe explain what is el. (el is the element reference) Thanks for your help. N/A N/A No response N/A No response
I think we can simplify the example to something like: And say "here's an example of assigning the result of a ternary operator to a variable. now do this for the elements text contents."
I started working on this and I was wondering how the localization is handled for the challenge markdowns? do I change one of the markdowns and run a script to apply it all?
You only need to update the English markdown. The rest are handled through Crowdin and our automated syncs. However, this issue has not been marked as open for contribution yet.
yeah I realized the issue is marked as discussing still, what more is needed to move the issue forward?
No one else has chimed in, so I'll go ahead and open it for contribution. A PR which resolves this issue should apply my suggestion:
Change:
}; ``` <del> When the `#search-input` element contains the value `Pikachu` and the `#search-button` element is clicked, the values in the `#pokemon-name`, `#pokemon-id`, `#weight`, `#height`, `#hp`, `#attack`, `#defense`, `#special-attack`, and `#special-defense` elements should be `PIKACHU`, `#25` or `25`, `Weight: 60` or `60`, `Height: 4` or `4`, `35`, `55`, `40`, `50`, `50`, and `90`, respectively. </del> <ins> When the `#search-input` element contains the value `Pikachu` and the `#search-button` element is clicked, the values in the `#pokemon-name`, `#pokemon-id`, `#weight`, `#height`, `#hp`, `#attack`, `#defense`, `#special-attack`, `#special-defense`, and `#speed` elements should be `PIKACHU`, `#25` or `25`, `Weight: 60` or `60`, `Height: 4` or `4`, `35`, `55`, `40`, `50`, `50`, and `90`, respectively. </ins> ```js async () => {

Discussion:
Example: When the #search-input element contains the value Pikachu and the #search-button element is clicked, the values in the #pokemon-name, #pokemon-id, #weight, #height, #hp, #attack, #defense, #special-attack, and #special-defense elements should be PIKACHU, or 25, Weight: 60 or 60, Height: 4 or 4, 35, 55, 40, 50, 50, and 90, respectively. here #speed is missing leading to unclear errors same holds true for the Gangar test case this could easily be fixed by changing the wording in: I excpected the test-case to include all elements that should've been return/changed 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
Change:
console.log(Date.now()); // 1628586800000 ``` <del> To see the new result, click on the `"Add New Task"` button. Then add a title of `WALK DOG` and click on the `"Add Task"` button. Open up the console to see the result with a random number added. </del> <ins> To see the new result, click on the `"Add New Task"` button. Then add a title of `WALK DOG` and click on the `"Add Task"` button. Open up the console to see the result. </ins> # --hints--

Discussion:
Wording in the description suggests a random number will appear in the console. The number generated is not random. It is a result of the calculation of the number of millisecond elapsed since January 1, 1970 00:00:00 UTC Remove the following text: On a new line, the following text to appear: ! 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
Hi, Can you mention the files where this change should be made.
This could be simplified to say "Open up the console to see the result"
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 (()); // ` Happy coding
Change:
Вы можете помочь нам: <del> - [📝 В исследовании, написании и обновлении статей нашего руководства.](#research-write-and-update-our-guide-articles) </del> <ins> - [📝 В исследовании, написании и обновлении статей нашего руководства.](#изучение-написание-и-обновление-статей-нашего-руководства) </ins> <del> - [💻 В создании, обновлении и исправлении ошибок в наших задачах по программированию.](#create-update-and-fix-bugs-in-our-curriculum-challenges) </del> <ins> - [💻 В создании, обновлении и исправлении ошибок в наших задачах по программированию.](#создание-обновление-и-исправление-ошибок-в-наших-задачах-на-кодирование) </ins> <del> - [🌐 В переводе обучающих статей и задач по программированию.](#translate-guide-articles-and-curriculum-challenges) </del> <ins> - [🌐 В переводе обучающих статей и задач по программированию.](#перевод-статей-руководства-и-задач-на-кодирование) </ins> <del> - [🛠 В исправлении ошибок на обучающей платформе freeCodeCamp.org](#help-us-fix-bugs-in-freecodecamporgs-learning-platform) </del> <ins> - [🛠 В исправлении ошибок на обучающей платформе freeCodeCamp.org](#помогите-нам-исправить-ошибки-в-обучающей-платформе-freecodecamporg) </ins> ### Изучение, написание и обновление статей нашего руководства

Discussion:
Describe your problem and how to reproduce it: Anchor links such is not working properly in several languages. For sure is not working in: Russian, Spanish, Chinese languages. Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Chrome Browser Version: Version 74.0.3729.169 (Official Build) (64-bit) Operating System: Windows 10 Pro 1809 If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box): !
These 3/4 PRs (exlude one - which I PR'd to my own forked-repo) may close this issue. Please review them.