jiebi/CodeConvo huggingface dataset:
Change:
stroke: var(--color-quaternary); } <del> .map-title svg:first-child { transform: rotate(90deg); } </del> .open > .map-title svg:first-child { <del> transform: rotate(-90deg); </del> <ins> transform: rotate(90deg); </ins> } .map-challenges-ul {

Discussion:
Describe the bug The arrow to the left of Expand courses/Collapse courses buttons points down when the list is in collapsed state and points up when the list is in expanded state. This is opposed to standard behavior of similar features in other apps. To Reproduce Steps to reproduce the behavior: to at the arrow to the left of Collapse courses. It points up while the list is in expanded state. on the above mentioned button to collapse the list. the list is collapsed state but the arrow points down. Expected behavior By convention, the arrow should point to the right or up when the list is in collapsed state and point down when the list is in expanded state. Screenshots ! ! Desktop: OS: Ubuntu 20.04.2 LTS Browser: Opera Version: 76 Additional context Also tested this on the same OS with Firefox 88.0.1 with the same result.
I agree, we should have the arrow pointing to the left when the courses are collapsed and down when they are expanded.
Hi Ahmad, I think you mean we should have the arrow pointing to the right when the courses are collapsed, right?
Hi. I'm new to contributing to FreecodeCamp, and just read the guidelines. With your permission, I will fork the project and try to solve the issue with a pull request.
Change:
isSignedInSelector, userSelector, challengeTestsSelector, <ins> isChallengeCompletedSelector, </ins> ( canFocus: boolean, { challengeType }: { challengeType: number },

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:
"Choose Node.js in the selection area below the name field.", "Click the Create button. Then click into your new workspace.", "In the lower right hand corner you should see a terminal window. In this window use the following commands. You don't need to know what these mean at this point.", <del> "Install <code>how-to-npm</code> with this command: <code>npm install -g git-it</code>", </del> <ins> "Install <code>git-it</code> with this command: <code>npm install -g git-it</code>", </ins> "Now start the tutorial by running <code>git-it</code>.", "Note that you can resize the c9.io's windows by dragging their borders.", "Make sure that you are always in your project's "workspace" directory. You can always navigate back to this directory by running this command: <code>cd ~/workspace</code>.",

Discussion:
Challenge has an issue. Please describe how to reproduce it, and include links to screenshots if possible. step 9 says Install how-to-npm with this command: npm install -g git-it. it looks like we are installing something called git-it instead of how-to-npm. As before, if I am incorrect please disregard this issue and just close. Just trying to do what I can for you guys. Love this site!
Hey thanks for posting an issue! It looks like you are right! I believe the step should say
That's what it looked like to me to. It seems that the command was copied from the next page when you do actually have to install how-to-npm. Btw, now that I have done a lot of the free code camp challenges, how would I go about starting to contribute to the project? I would love to help out with the code or issues or anything really.
Awesome! Just jump right in :smile:! Maybe start with putting in a pull request for this issue? Feel free to ping me in gitter if you want to chat about this further (so as to not pollute this issue with conversation)
Dear camper, your issue is related to: Please see .
I checked when I was creating the pull request to fix this issue and it seems that the how-to-npm part of it is already fixed.
Change:
); ``` <del> `knapsackUnbounded([{ name:"panacea", value:3000, weight:0.3, volume:0.025 }, { name:"ichor", value:1800, weight:0.2, volume:0.015 }, { name:"gold", value:2500, weight:2, volume:0.002 }], 35, 0.35)` should return `75300`. </del> <ins> `knapsackUnbounded([{ name:"panacea", value:3000, weight:0.3, volume:0.025 }, { name:"ichor", value:1800, weight:0.2, volume:0.015 }, { name:"gold", value:2500, weight:2, volume:0.002 }], 35, 0.35)` should return `75900`. </ins> ```js assert.equal(

Discussion:
Describe your problem and how to reproduce it: One of the tests in this challenge has wrong result, both in test code and description. It says: This isn't correct. For example: knapsack having 11 panacea, 3 ichor and 15 gold has value , while being within the weight and value constrains: This seems to be case of precision issue. Multiplication input item weights, volumes and max values for those parameters, to start operations without decimals, makes result as it should be, also for the problematic test. Add a Link to the page with the problem:
Yes, it seems you are right - do you want to propose a solution?
Yes, sorry. The least invasive way to correct the seems to be to multiply in function, by the same factor, initial items weights, volumes, and in the first recursive call multiply and . This is enough to make calculations on integers and get correct answers. I don't know if just such change would be okay or if more desirable would be more elaborate rewrite of solution. Or at least changing to / and renaming some variables to better names.
would you be willing to open a PR for it so that the challenge can be changed? you can also contribute to the guide by opening a topic in the #subforum category in the forum
Definitely yes, although I need advice which approach here would be preferred. Considering that the sample solution in code is just used for internal testing and just small change to it can fix the issue.
Change:
# --description-- <del> Promises are most useful when you have a process that takes an unknown amount of time in your code (i.e. something asynchronous), often a server request. When you make a server request it takes some amount of time, and after it completes you usually want to do something with the response from the server. This can be achieved by using the `then` method. The `then` method is executed immediately after your promise is fulfilled with `resolve`. Here’s an example: </del> <ins> Promises are most useful when you have a process that takes an unknown amount of time in your code (i.e. something asynchronous), often a server request. When you make a server request it takes some amount of time, and after it completes you usually want to do something with the response from the server. This can be achieved by using the `then` method. ```js Promise.prototype.then(onFulfilled, onRejected) ``` The `then` method schedules callback functions for the eventual completion of a Promise - either fulfillment or rejection. One of the `onFulfilled` and `onRejected` handlers will be executed to handle the current promise's fulfillment or rejection. When the promise is fulfilled with `resolve` the `onFulfilled` handler is called. </ins> ```js myPromise.then(result => {

Discussion:
I realize it is probably just an oversimplification but the part that may need correction is. I think this should say the handler is called when the promise becomes fulfilled. Or something to that effect. More precise verbiage. 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] Forum: Specs/MDN:
I'd say something like: I think mentioning the might get confusing.
The thing is, it takes two callbacks. An and the optional . So we may have to make the distinction. The name is just the spec definition. It might be less confusing if we show the signature. I pretty much just stole the text from MDN. I think we are far enough into the curriculum that a little technical lingo isn't the worst. It might be a bit verbose although it explains it better if you ask me.
I could go with that, yeah.
Change:
"<span class='text-info'>Bonus User Story:</span> As a user, I navigate to different sections of the webpage by clicking buttons in the navigation.", "Don't worry if you don't have anything to showcase on your portfolio yet - you will build several apps on the next few CodePen challenges, and can come back and update your portfolio later.", "There are many great portfolio templates out there, but for this challenge, you'll need to build a portfolio page yourself. Using Bootstrap will make this much easier for you.", <del> "Note that CodePen.io overrides the Window.open() function, so if you want to open windows using jquery, you will need to target invisible anchor elements like this one: <code>&lt;a target='_blank'&gt;</a></code>.", </del> <ins> "Note that CodePen.io overrides the Window.open() function, so if you want to open windows using jQuery, you will need to target invisible anchor elements like this one: <code>&lt;a target='_blank'&gt;</a></code>.", </ins> "Remember to use <a href='//github.com/FreeCodeCamp/freecodecamp/wiki/How-to-get-help-when-you-get-stuck' target='_blank'>Read-Search-Ask</a> if you get stuck.", "When you are finished, click the "I've completed this challenge" button and include a link to your CodePen. If you pair programmed, you should also include the Free Code Camp username of your pair.", "If you'd like immediate feedback on your project from fellow campers, click this button and paste in a link to your CodePen project. <br><br><a class='btn btn-primary btn-block' href='https://twitter.com/intent/tweet?text=Check%20out%20the%20project%20I%20just%20built%20with%20%40FreeCodeCamp:%20PASTE_YOUR_CODEPEN_URL_HERE%20%0A%20%23LearnToCode%20%23JavaScript' target='_blank'>Click here then add your link to your tweet's text</a>"

Discussion:
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.86 Safari/537.36</code. Please describe how to reproduce this issue, and include links to screenshots if possible. Step 13 should have the "Q" capitalized in jQuery. !
Change:
## --text-- <del> Which of the following options can be used set a breakpoint in Visual Studio Code? </del> <ins> Which of the following options can be used to set a breakpoint in Visual Studio Code? </ins> ## --answers--

Discussion:
Hello, On the challenge project "Debug a C# Console Application Using Visual Studio Code, the question is: The word is missing between and . N/A I believe the question should be: No response Device: all OS: all Browser: all Version: all 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.
Change:
const origin = returnUrl.origin; // if this is not one of the client languages, validation will convert // this to '' before it is used. <del> const pathPrefix = returnUrl.pathname.split('/')[0]; </del> <ins> const pathPrefix = returnUrl.pathname.split('/')[1]; </ins> return _normalizeParams({ returnTo: returnUrl.href, origin, pathPrefix }); }

Discussion:
If I am signed in as a user while on and log out, I am redirected to the english home page. redirects to the chinese page.
wrong endpoint in server code? Thats what I would assume, but am not sure how to locate where the correct endpoint would be placed in the code.
Here is the endpoint: and most likely the issue is with the redirect "predictor" utility: However we would want to confirm the cause before we open this up for contributions.
yes, that's the issue. To fix it we can grab the from and combine that with to get the desired redirect.
Should that be done in the getRedirectParams function or should the path be put together from the data set returned by getRedirectParams in
You need to construct the URL from the values you get from the as a consumer of the function and not change its existing behavior. In simple terms the change is required within the endpoint handler.
That makes sense.
I got the codebase installed locally is there a way to run just
Not sure what you mean by running only one file? The API server is an express based application (Loopback 3). You need to start up at least the api-server app. If you want to do that, you should be able to do that by navigating to directory and using the run-script commands in its
I'm trying to figure out what gets passed in pathPrefix, from getRedirectParams, when one signs out from the Spanish page. But when running the site locally, switching to Espanol leads to a "This site can't be reached" error. Just for clarity's sake, I realize now that is not meant to be run alone and I am using the "npm run develop" command to try and recreate this bug locally. Is this a bad approach?
Ah - right. Sorry about sending you on a goose-chase. When you build locally, we do not have a way yet to develop both and at the same time alongside the API app. I am going to untag this from help-wanted and assign this instead. Oliver, you should be able to test / reproduce and recommend a fix on our staging instance - not sure because this may be needing access for "close to prod" type setup.
I was just in the process of going through the steps, when you commented Mrugesh!

I'll leave them here for the curious: Unfortunately testing locally is quite a pain, because we use since is entirely separate from . It's a bit of a slow, memory intensive process, but you can do this: in your .env file Then you can go to
Change:
"Here's the anatomy of a CSS class:", "<img class='img-responsive' alt='a diagram of how style tags are composed, which is also described in detail on the following lines.' src='https://www.evernote.com/l/AHSCzZV0l_dDLrqD8r9JsHaBWfEzdN0OpRwB/image.png'/>", "You can see that we've created a CSS class called "blue-text" within the <code>&#60;style&#62;</code> tag.", <del> "You can apply a class to an HTML element like this: <code>&#60;h2 class="blue-text"&#62;CatPhotoApp&#60;h2&#62;</code>", </del> <ins> "You can apply a class to an HTML element like this: <code>&#60;h2 class="blue-text"&#62;CatPhotoApp&#60;/h2&#62;</code>", </ins> "Note that in your CSS <code>style</code> element, classes should start with a period. In your HTML elements' class declarations, classes shouldn't start with a period.", "Instead of creating a new Style tag, try removing the h2 style declaration from the existing style element, and replace it with the class declaration for ".red-text"." ],

Discussion:
Challenge has an issue. Please describe how to reproduce it, and include links to screen shots if possible. What I see: You can apply a class to an HTML element like this: <h2 class="blue-text"CatPhotoApp<h2What I expected to see: You can apply a class to an HTML element like this: <h2 class="blue-text"CatPhotoApp</h2
Markdown's getting in the way, see screenshot. !
Change:
"Wait for the workspace to finish processing and select it on the left sidebar, below the Create New Workspace button.", "Click the "Start Editing" button.", "In the lower right hand corner you should see a terminal window. In this window use the following commands. You don't need to know what these mean at this point.", <del> "Run this command: <code>npm install how-to-npm -g</code>", "Now start this tutorial by running <code>npm install how-to-npm@2.0.0</code>.", </del> <ins> "Install <code>how-to-npm</code> with this command: <code>npm install how-to-npm -g</code>", "Now start the tutorial by running <code>how-to-npm</code>.", </ins> "Note that you can resize the c9.io's windows by dragging their borders.", "Make sure that you are always in your project's "workspace" directory. You can always navigate back to this directory by running this command: <code>cd ~/workspace</code>.", "Note that you can only add dist tags to the specific version numbers published in steps 8 and 10. If you receive a 403 or 404 error, run <code>how-to-npm</code> and try again.",

Discussion:
One of the instructions has the following: start this tutorial by running npm install how-to-npm it says to run the npm tutorial by typing 'npm install how-to-npm but you are actually supposed to just type 'how-to-npm'
Change:
```yml tests: - text: Your code should add one <code>s</code> tag to the markup. <del> testString: assert($('s').length == 1, 'Your code should add one <code>s</code> tag to the markup.'); </del> <ins> testString: assert($('s').length == 1); </ins> - text: A <code>s</code> tag should wrap around the Google text in the <code>h4</code> tag. It should not contain the word Alphabet. <del> testString: assert($('s').text().match(/Google/gi) && !$('s').text().match(/Alphabet/gi), 'A <code>s</code> tag should wrap around the Google text in the <code>h4</code> tag. It should not contain the word Alphabet.'); </del> <ins> testString: assert($('h4 > s').text().match(/Google/gi) && !$('h4 > s').text().match(/Alphabet/gi)); </ins> - text: Include the word Alphabet in the <code>h4</code> tag, without strikethrough formatting. <del> testString: assert($('h4').html().match(/Alphabet/gi), 'Include the word Alphabet in the <code>h4</code> tag, without strikethrough formatting.'); </del> <ins> testString: assert($('h4').html().match(/Alphabet/gi)); </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: The test case is passed though I have not the s tag around the word Google in the h4 tag, it is counting the s tag around Google word in the p tag as valid. Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Google Chrome Browser Version: Version 74.0.3729.131 (Official Build) (64-bit) Operating System: Windows 10 64-bit If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box): !
Thank you for reporting this bug. I can confirm it needed to be fixed and I have created a pull request () which should resolve the issue.
Change:
- text: You should remove the ES5 assignment syntax. testString: assert(!code.match(/highToday = HIGH_TEMPERATURES.today/g) && !code.match(/highTomorrow = HIGH_TEMPERATURES.tomorrow/g)) - text: You should use destructuring to create the <code>highToday</code> variable. <del> testString: assert(code.match(/(var|const|let)s*{s*(today:s*highToday[^}]*|[^,]*,s*todays*:s*highTodays*)}s*=s*HIGH_TEMPERATURES(;|s+|//)/g)); </del> <ins> testString: assert(code.match(/(var|const|let)s*{s*(todays*:s*highToday[^}]*|[^,]*,s*todays*:s*highTodays*)}s*=s*HIGH_TEMPERATURES(;|s+|//)/g)); </ins> - text: You should use destructuring to create the <code>highTomorrow</code> variable. <del> testString: assert(code.match(/(var|const|let)s*{s*(tomorrow:s*highTomorrow[^}]*|[^,]*,s*tomorrows*:s*highTomorrows*)}s*=s*HIGH_TEMPERATURES(;|s+|//)/g)); </del> <ins> testString: assert(code.match(/(var|const|let)s*{s*(tomorrows*:s*highTomorrow[^}]*|[^,]*,s*tomorrows*:s*highTomorrows*)}s*=s*HIGH_TEMPERATURES(;|s+|//)/g)); </ins> ``` </section>

Discussion:
In this challenge ,if i use space between , the following ouptut genrates. (code) (output) However,if i remove the space , there is no error in the console. So i crosscheck this problem in my local system and found that space does't generate the error.
. It should allow a space there with the following... assert((/(var//)/g)); The code that needs to be is highlighted.
Change:
The code editor has most of the `MyToDoList` component set up. Some of this code should look familiar if you completed the controlled form challenge. You'll notice a `textarea` and a `button`, along with a couple of methods that track their states, but nothing is rendered to the page yet. <del> Inside the `constructor`, create a `this.state` object and define two states: `userInput` should be initialized as an empty string, and `toDoList` should be initialized as an empty array. Next, delete the comment in the `render()` method next to the `items` variable. In its place, map over the `toDoList` array stored in the component's internal state and dynamically render a `li` for each item. Try entering the string `eat, code, sleep, repeat` into the `textarea`, then click the button and see what happens. </del> <ins> Inside the `constructor`, create a `this.state` object and define two states: `userInput` should be initialized as an empty string, and `toDoList` should be initialized as an empty array. Next, delete the `null` value in the `render()` method next to the `items` variable. In its place, map over the `toDoList` array stored in the component's internal state and dynamically render a `li` for each item. Try entering the string `eat, code, sleep, repeat` into the `textarea`, then click the button and see what happens. </ins> **Note:** You may know that all sibling child elements created by a mapping operation like this do need to be supplied with a unique `key` attribute. Don't worry, this is the topic of the next challenge.

Discussion:
Instructions say: "Next, delete the comment in the method..." instead of: "Next, delete the value in the method..."