jiebi/CodeConvo huggingface dataset:
Change:
max-width: 600px; } <ins> .challenge-description-header { display: flex; } </ins> .description-container h1 { color: var(--secondary-color); font-family: Roboto Mono, monospace;

Discussion:
I am learning from the RWD course, and I wanted to revise what I had done previously. I went to previous steps, but I found that the steps are shown incomplete when I open them. It tells you to do the step again. This doesn't make sense because you have a button at the top if you do want to do the step again. Completed code should be shown in completed challenge's code editor along with an indicator like ! !
It would be relatively easy to show the solution to a step, and the "Step completed" text, (rather than storing the user's code) if a user has already submitted that step. If other maintainers are in favour of this change, I'd be happy to open this up for PRs.
My concern with that approach would be it completely destroys the UX if someone wants to go through a practise project again.
I propose to add a button under the summary of each project for this issue.
I'm not keen on adding a button to do that, given the potential for frustration if someone triggers that accidentally.
I think I'd be in favour of the existing flow, presonally.
At a minimum, we could probably add the checkmark next to "Step 22" like we do for all the other challenges after it has been completed. That seems to be missing here. !
That's a good idea, IMO.
I think just adding that check mark is probably the way to go then. It keeps it consistent with everything else. I'm going to open it up for contributions. If anyone still has reservations, be sure to bring them up.
May I suggest in addition to indicating completed challenges in the challenge itself, to do that in the list of steps. Here is a crude representation of what I have in mind: <p align="center"<img src="https://user-" width=500 align="centre"/</p
I believe it already does that. Here's a screenshot of what the challenge looks like for me, after I've completed it (the steps are highlighted in blue instead of white). !
Also, I would like to work on this issue, and I agree that adding a checkmark next to the step would be a good fix, but I noticed that the RWD courses don't have a step number or title at the top like the other courses do.

For other courses, the checkmark for completion is displayed next to the name of the challenge, like you can see in the first screenshot (of the JS Algorithms & Data Structures courses). However, for the RWD course, there is nothing in this location, as you can see in the second screenshot. ! ! How should we go about adding a checkmark to the RWD course? Should we just put it next to where it says on the screenshot (inside the instructions block)? Or should we move the step number to the same place as where the challenge names are for the other courses and put the checkmark there, in order to be consistent with the other courses?
Thanks I could not log in since yesterday and this is probably the reason I don't see indication for completed challenges in the list of steps.
Change:
Your code should use `then` to handle the data converted to JSON by the other `then`. ```js <del> assert(code.match(/.thens*(s*(data|(s*datas*))s*=>s*{[^}]*}s*)/g)); </del> <ins> assert(__helpers.removeWhiteSpace(code).match(/.then((?w+)?=>{[^}]*})/g)); </ins> ``` Your code should get the element with id `message` and change its inner HTML to the string of JSON data. ```js assert( <del> code.match( /documents*.getElementByIds*(s*('|")message1s*)s*.innerHTMLs*=s*JSONs*.s*stringifys*(s*datas*)/g </del> <ins> __helpers.removeWhiteSpace(code).match( /document.getElementById(('|")message1).innerHTML=JSON.stringify(?w+)/g </ins> ) ); ```

Discussion:
Challenge Page: File: Code which passes: Code which fails: The test should not rely on the variable name. Suggestion for 3rd test: An issue with the help wanted or first timers only label is open 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
I would love to work on this Please assign this to me
Hey Usually if there is a 'Help Wanted' label, anyone can start working on it. I think meant to do this, but You can definitely start working on this! Thank you for helping us improve freeCodeCamp!
No problem I will wait for some time for Shaun to do it. ELse I will start working on it
Hey there, This is open for contribution. Please read the last paragraph of the original post.
Since there's a 'first timers only' label, you're very welcome to contribute
Change:
name, user: { username } } = props; <del> const { challengeFiles, isOpen, projectTitle, solution } = solutionState; </del> <ins> const { files, isOpen, projectTitle, solution } = solutionState; </ins> return ( <div> {t(

Discussion:
When trying to view a JS solution, the following error appears in dev, explaining the issue in production. It seems we followed the expected prop-types too early (ref: ). The object is still an object literal. Reported on forum:
I have the same problem as I said in the forum, the bottom error says it responded with 404 to the request for the certification on JS algorithms and data structures. Seems like there's an issue with the backend reaching that certification. !
Change:
(getUserInput) => $.get(getUserInput('url') + '/_api/get-tests?type=functional&n=5').then( (data) => { <del> assert.equal(data.assertions[3].method, 'browser.element'); </del> <ins> assert.equal(data.assertions[3].method, 'browser.elements'); </ins> assert.match(data.assertions[3].args[0], /('|")span#dates1/); assert.equal(data.assertions[3].args[1], 1); },

Discussion:
<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --Affected page Add a link to the coding challenge with the problem. Your code Copy and paste the code from the editor that you used in between the back-ticks below: Expected behavior Add a clear and concise description of what you expected to happen. The code should pass the fcc tests Screenshots If applicable, add screenshots to help explain your problem. you can drag and drop, png, jpg, gif, etc. in this box. ! System (please complete the following information): Irrelevant Additional context Add any other context about the problem here. The solution at uses ( s missing ) and the tests are using that. It is inconsistent with the previous lesson, and according to the Zombie docs, we could either use ('span#name') or ('span#name', 1)
Hello there, Thank you, for taking the time to open this. It appears this was missed with the previous PR: , and issue. As such, it is a simple fix to correct the test at: So, I am going to open this up to first timers An issue with the help wanted or first timers only label is open 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
Change:
assert.equal(document.querySelector('form')?.action, 'https://register-demo.freecodecamp.org/'); ``` <ins> Your `form` element should have a closing tag `</form>`. ```js assert.match(code, /</form>/); ``` </ins> # --seed-- ## --seed-contents--

Discussion:
The form element is missing a closing tag, but the challenge still accept it and let me move onto the next step It should respond with "You are missing the closing tag", instead of accepting it and letting you move onto the next step ! 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] According to It should result in a valid form element, because the browser will close it. However, the element technically does not have any "Tag omission" so it should have an end tag in the HTML (even if it gets to the DOM by the browser).
This is no more the first project, it would become regex galore, and when possible instead we prefer to not use regex, so fixing all these tests is probably not worth it
We don't need to keep testing for closing tags everywhere. But this challenge is about building a form, so having one test to check the initial form element is created correctly doesn't seem unreasonable.
Agreed, this seems like a good reason to regex test a closing tag. Opening for contribution.
Change:
# --description-- <del> Inside the `button`, create two more `span` elements. The first span element should have class `playlist-song-artist` and interpolate `song.artist`. The second span element should have class `playlist-song-duration` and interpolate `song.duration`. </del> <ins> Inside the `button` element, create two more `span` elements. The first `span` element should have a `class` of `playlist-song-artist`. In between the `span` tags, add `${song.artist}`. The second `span` element should have a `class` of `playlist-song-duration`. In between the `span` tags, add `${song.duration}`. </ins> # --hints--

Discussion:
This should be a clearer description and fix the inconsistencies for the user of backticks. ` 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
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 Happy coding
Change:
"  // Only change code below this line.", "", "", <ins> "", "  // Only change code above this line.", </ins> "}", "", <del> "// Only change code above this line.", </del> "// We use this function to show you the value of your variable in your output box.", "(function(){return myFunction();})();" ],

Discussion:
Challenge has an issue. the "code space" where students are meant to put their if/else code contains (by default) a closing bracket which is part of the test code and shouldn't be edited/removed: <a href="" target="_blank"<img src="" border="0" alt=" photo "/</aI think this closing bracket should be moved under the comment to avoid students using it inside their if/else statement (and breaking the function). (someone was stuck on this earlier today)
Thank you for reporting! :)
Change:
"assert($("a").text().match(/catsphotos/gi), 'Your <code>a</code> element should have the anchor text of "cat photos"')", "assert($("p") && $("p").length > 2, 'Create a new <code>p</code> element around your <code>a</code> element.')", "assert($("a[href="http://www.freecatphotoapp.com"]").parent().is("p"), 'Your <code>a</code> element should be nested within your new <code>p</code> element.')", <del> "assert($("p").text().match(/^s*Viewsmores/gi), 'Your <code>p</code> element should have the text "View more " (with a space after it).')", </del> <ins> "assert($("a[href="http://www.freecatphotoapp.com"]").parent().text().match(/^s*Viewsmores/gi), 'Your <code>p</code> element should have the text "View more " (with a space after it).')", </ins> "assert(!$("a").text().match(/Viewsmore/gi), 'Your <code>a</code> element should <em>not</em> have the text "View more".')", "assert(editor.match(/</p>/g) && editor.match(/<p/g) && editor.match(/</p>/g).length === editor.match(/<p/g).length, 'Make sure each of your <code>p</code> elements has a closing tag.')", "assert(editor.match(/</a>/g) && editor.match(/<a/g) && editor.match(/</a>/g).length === editor.match(/<a/g).length, 'Make sure each of your <code>a</code> elements has a closing tag.')"

Discussion:
The following code should pass, but won't because the <ptag is at the bottom, even though the challenge doesn't require it to be. Please refactor the tests to allow this situation to pass. Challenge has an issue. User Agent is: <codeMozilla/5.0 (Macintosh; Intel Mac OS X ) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2490.80 Safari/537.36</code. Please describe how to reproduce this issue, and include links to screenshots if possible. My code:
I just arranged all the html elements, it accepted and changed 'View more ' to "Click her for " .......

Challenge has an issue. User Agent is: <codeMozilla/5.0 (X11; Linux x86_64; rv:41.0) Gecko/ Firefox/41.0</code. Please describe how to reproduce this issue, and include links to screenshots if possible. My code: In the code above when I wrote the line: " <pView more <a href=""cat photos</a</p" at the end.It did not made all the problems correct. But when I used the same code just above the cat picture it showed correct output. Attaching the screen shot for wrong output(line 38): ! Attaching the screen shot for correct output(line 30): ! There's no difference in the code except the position(line 38 in 1st screen shot vs line 30 in 2nd screen shot).
This test expects the wrapping tag you insert to be the first tag on the page. This had been fixed by a previous PR that has been overwritten. Someone will need to make a new PR that either makes it clear where the user needs to put their code or makes the test not care.
Yes,that should be made clear because otherwise wrong result for this is frustrating.
Change:
import './welcome.css'; <ins> // created outside the function, so that the quotes don't change unless the // page is reloaded. const { quote, author } = randomQuote(); </ins> function Welcome({ name }) { <del> const { quote, author } = randomQuote(); </del> return ( <Fragment> <Row>

Discussion:
Describe the bug When you left click while on the page the random quote gets replaced with a new one. To Reproduce Steps to reproduce the behavior: to on anything that doesn't take you to a new page the random quote get replaced Expected behavior A random quote should appear each time you visit learn, but it should not change unless you reload. Screenshots ! Additional Context This only appears once the site has been built, doesn't show the error. It is not happening in production, so presumably was introduced since
I see this on the production site as well.
Change:
/> ) : null} <SignoutModal /> <del> {isChallenge && ( </del> <ins> {isChallenge && !showCodeAlly && ( </ins> <div className='breadcrumbs-demo'> <BreadCrumb block={block as string}

Discussion:
The page gets stuck and goes blank with you leave the relational database challenges. to 'Start the course' the fCC logo/button in the nav goes blank and nav disappears Page not to go blank. Clicking fCC logo should go to /learn No response Same on Chrome and Firefox Console errors: <img width="671" alt="Screen Shot 2023-02-16 at 11 44 19 AM" src="https://user-"
Change:
You should set the `color` property to `grey`. ```js <del> const hasColor = new __helpers.CSSHelp(document).getCSSRules().some(x => x.style.color === 'grey'); </del> <ins> const hasColor = new __helpers.CSSHelp(document).getCSSRules().some(x => (x.style.color === 'grey' || x.style.color === 'gray')); </ins> assert(hasColor); ```

Discussion:
I set the a:visited color to “gray” which changed the link color in the preview pane and even displayed the color picker box in the view with the color gray but when I clicked Check Your Code it didn’t accept my answer. I had to change it to “grey” with an ‘e’, not an ‘a’ to get the Step completed message. Now it does have the word “grey” with an ‘e’ in the directions, but I spell gray with an ‘a’ so that is what I did naturally. Do you want to accept a value of “gray” and “grey” as correct answers or will you only accept “grey”. If you only accept “grey” would it be possible to add a message asking the user to check their spelling? Otherwise, users may get frustrated and confused if they used “gray” and then they don’t understand why the answer is wrong. ! <!-- Please complete the following information. --- Device: [Laptop] OS: [Windows 10] Browser: [brave] Version: [Version 1.33.106 Chromium: 96.0.4664.110 (Official Build) (64-bit)] <!-- Add any other context about the problem here. --
Change:
--fcc-editable-region-- --fcc-editable-region-- <del> if (done === count) { </del> <ins> if (done === count) { </ins> continueLoop = false; } }

Discussion:
In the JS Beta project to build a pyramid gen., steps 82 and 83 both show an if statement block in the code editor that is not correctly indented within the while loop it is in. This may confuse new learners if they are relying on the indents to understand the logic and should be fixed to keep the presented code in acceptable shape for learners. N/A the if statement should be indented within the while loops block ! ! N/A No response
Change:
margin-left: 20px; margin-right: 20px; } <ins> .underlined-link { text-decoration: underline; } </ins> @media (min-width: 991px) and (max-width: 1199px) { .testimonial-copy { height: 150px;

Discussion:
I can work on this. A little confused though, since it appears to be there already. Does the mission statement go here? Please provide instructions, thanks.
Change:
id: 587d7b7e367417b2b2512b24 title: Use the Conditional (Ternary) Operator challengeType: 1 <del> videoUrl: 'https://scrimba.com/c/c3JRmSg' </del> forumTopicId: 301181 ---

Discussion:
The scrimba video for needs to be updated to match the updates made a while back.
Any update on this?
I want to do this kind of update, but I am not sure what accurate need to be updated. Could anyone explain more about that. Thanks.
Were you able to correct the video?
If there's no movement on creating a new video to update this, I'd be happy to take a crack at it.
I checked out the video and it seemed to be factually accurate. Is the video update just needing to match the condition of "Equal" or "Not Equal" like the actual problem is asking for?
Change:
value={this.getUserEmail()} /> </FormGroup> <del> <StripeCardForm getValidationState={this.getValidationState} /> </del> <ins> <StripeCardForm getValidationState={this.getValidationState} theme={theme} /> </ins> <Button block={true} bsStyle='primary'

Discussion:
Describe the bug ! The input text on the darkmode version of the donate page () are not readable so you cannot see what you are typing. To Reproduce Steps to reproduce the behavior: to '' with darkmode enabled. on the input labled 'Your Card Number:' and enter some numbers. entered input is not readable as the color of the input text is the same as the background-color of the input. bug applies to the credit card expiration and CVC inputs. ! Desktop (please complete the following information): OS: Win10 Browser Brave/Chrome/Firefox
Change:
"tests": [], "type": "waypoint", "challengeType": 3, <del> "nameEs": "Crea una aplicación de votaciones" </del> <ins> "nameEs": "Desafios para aprender D3", "descriptionEs": [] </ins> } ] }

Discussion:
[x] Learn D3 Challenges
what should we do with these ones? I guess they are not finished, so there's no point in translating that one line, or am I wrong?
Change:
</Button> </> )} <del> {challengeType !== challengeTypes.multifilePythonCertProject && ( <> <Spacer size='xxSmall' /> <Button block={true} variant='primary' onClick={openResetModal}> {isMobile ? t('buttons.reset') : t('buttons.reset-lesson')} </Button> </> )} </del> <ins> <> <Spacer size='xxSmall' /> <Button block={true} variant='primary' onClick={openResetModal}> {isMobile ? t('buttons.reset') : t('buttons.reset-lesson')} </Button> </> </ins> <Spacer size='xxSmall' /> <Dropdown dropup> <Dropdown.Toggle

Discussion:
The "reset" button went missing on the projects editor Any page with older style editor The reset button should be present. All
There is no need for a reset button here, you can just remove all code?
The cash register starts with code
I think we should have a reset button for the odin remix projects If a camper is a working through one of those challenges, and messes up their HTML, then they should have the ability to reset it. But also, for the JS certs, sometimes it is easier to just click reset and start with a clean slate. Plus the other editors have the reset button, so we should have it for consistency sake IMO.
This seems like a great place to start investigating in my opinion. These seem to be multifile cert projects after all.
Reopening, python final projects still miss the button
Change:
}); createFiles( <del> mergeChallengeFiles(challengeFiles, savedChallenge?.challengeFiles) </del> <ins> mergeChallengeFiles(seedChallengeFiles, savedChallenge?.challengeFiles) </ins> ); initTests(tests);

Discussion:
During development adding or modifying the head or tail code ( and ) don't seem to be updated in the challenge. Tested locally and on gitpod. Steps to reproduce the behavior: /modify the or in challenge. For example one of the steps in the JS curriculum. I was adding function and variables to it, and then was attempting to use them in tests. and wait for challenge to be updated. () the challenge page, or visit it for the first time. running tests, referencing the function/variables from head/tail code, there will be reference errors. Changing head or tail code, should update the challenge to use it. Login status don't seem to affect issue. During changing the code, thinking I might do something incorrectly, there were handful times when challenge was updated with the head/tail code. Not sure what was the exact reason. What makes it stranger is stopping client develop and starting again, is not necessarily solving the problem. At some point I thought that's working to get challenge fully updated, but right now that doesn't seem to work.
I might have seen this as well. I remember trying to save a regex in to use in multiple tests, and it didn't seem to work. Kind of just thought I was using it incorrectly, but maybe this was the issue.
Thanks for the detailed investigation of another tricky bug. It was another example where saved code messes things up. They're always a pain to debug because everyone sees something different, depending on whether or not they have something saved. Hopefully this is the last of them!
What's the most throwing off, that wasn't even code saved manually!
Weird. Is there nothing in your local storage?
Hmm, well there is after running tests. That must have been the point when saving mishap happened.
Oh, that's right. No mystery beyond "how did I forget this?" :face_exhaling:
Change:
// remove additional border styles after migration const listClassNames = <del> 'flex mb-0 pl-0 mt-0  border-t-[0px] border-r-[0px] border-l-[0px] border-b-[1px] border-solid border-foreground-quaternary'; </del> <ins> 'flex mb-0 pl-0 mt-0 border-b-[1px] border-solid border-foreground-quaternary'; </ins> export const TabsTrigger = React.forwardRef< React.ElementRef<typeof Trigger>,

Discussion:
Remove the following classes from the component: - Code:
sorry for misunderstanding - what is the thing which need to be done?