jiebi/CodeConvo huggingface dataset:
Code change:
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

Related discussion:
text: export const devLoginRedirect = () => { return (req, res) => { <del> // this mirrors the production approach, but without any validation </del> <ins> // this mirrors the production approach, but only validates the prefix </ins> let { returnTo, origin, pathPrefix } = getRedirectParams( req, <del> params => params </del> <ins> ({ returnTo, origin, pathPrefix }) => { pathPrefix = availableLangs.client.includes(pathPrefix) ? pathPrefix : ''; return { returnTo, origin, pathPrefix }; } </ins> ); returnTo += isRootPath(getRedirectBase(origin, pathPrefix), returnTo) <del> ? 'learn' </del> <ins> ? '/learn' </ins> : ''; return res.redirect(returnTo); };
Code change:
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

Related discussion:
text: 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 }); }
Code change:
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.

Related discussion:
text: 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 {
Code change:
! there is written stUnes instead of stOnes This looks something that can be fixed by "first time" code contributors to this repository. Here are the files that you should be looking at to work on a fix: List of files: Please make sure you read , we prioritize contributors following the instructions in our guides. Join us in or if you need help contributing, our moderators will guide you through this. Sometimes we may get more than one pull requests. We typically accept the most quality contribution followed by the one that is made first. Happy contributing.

Related discussion:
text: A game is played with three piles of stones and two players. <del> On each player's turn, the player removes one or more stones from the piles. However, if the player takes stones from more than one pile, the same number of stunes must be removed from each of the selected piles. </del> <ins> On each player's turn, the player removes one or more stones from the piles. However, if the player takes stones from more than one pile, the same number of stones must be removed from each of the selected piles. </ins> In other words, the player chooses some $N > 0$ and removes:
Code change:
!
If possible please assign this issue to me! as I am new to open source
For future reference, it would help to have a link to the challenge when creating an issue It's To clarify, you want to change: to: That's not bad. What about
Yes, that is better! and sorry for any inconvenience as I am new to open source
Please, can you assign me to resolve that issue? That'll be a great start for my open source journey!
We do not assign issues The first PR that fixes the problem will be accepted. If you have not done so already, you should read the . Thanks for your interest in contributing If you have any questions, feel free to ask - or you can reach out in the
Please review the PR
please be patient, people will review when they can.
Alright!

Related discussion:
text: Text is often a large part of web content. CSS has several options for how to align it with the `text-align` property. <del> `text-align: justify;` causes all lines of text except the last line to meet the left and right edges of the line box. </del> <ins> `text-align: justify;` spaces the text so that each line has equal width. </ins> `text-align: center;` centers the text
Code change:
Hey everyone! On the lesson , with regards to and , it says: I think this is not actually correct. A more appropriate definition could be: Otherwise it may lead people to believe it's getting before/after the actual element. Another definition could be (taken from the MDN):
i'll do it (i like MDN definition)

Related discussion:
text: # --description-- <del> One of the most popular shapes in the world is the heart shape, and in this challenge you'll create one using pure CSS. But first, you need to understand the `::before` and `::after` pseudo-elements. These pseudo-elements are used to add something before or after a selected element. In the following example, a `::before` pseudo-element is used to add a rectangle to an element with the class `heart`: </del> <ins> One of the most popular shapes in the world is the heart shape, and in this challenge you'll create one using pure CSS. But first, you need to understand the `::before` and `::after` pseudo-elements. ::before creates a pseudo-element that is the first child of the selected element; ::after creates a pseudo-element that is the last child of the selected element. In the following example, a `::before` pseudo-element is used to add a rectangle to an element with the class `heart`: </ins> ```css .heart::before {
Code change:
The instruction looks kinda confusing. The word can be replaced by . ! <!-- Please complete the following information. --- Device: Laptop OS: Windows 10 Browser: firefox Version: 98.0.1 (64-bit) <!-- Add any other context about the problem here. --
Yeah - I see what you mean here. While it's implicit and really a nitpick, we can be clear as water in verbiage.

Related discussion:
text: The `fieldset` element is used to group related inputs and labels together in a web form. `fieldset` elements are <dfn>block-level elements</dfn>, meaning that they appear on a new line. <del> Nest the `Indoor` and `Outdoor` radio buttons within a `fieldset` element, and don't forget to indent the buttons. </del> <ins> Nest the `Indoor` and `Outdoor` radio buttons within a `fieldset` element, and don't forget to indent the radio buttons. </ins> # --hints--
Code change:
The title for code preview modals is left aligned. The rest of our modals have a centered title. to on view -show code left aligned title it -click view -show project is centered Center the title Code modal: ! Preview modal: ! I think I would just center the title, not the HTML/CSS/JS title
I wonder how should I tackle these, If I can tackle it of course. there is there solution: adding inline CSS to the adding with value to file we can go wild and change value to include
I would look at how the other titles are getting centered. Maybe there's just a class to those we didn't add on the others or something.
I see there is with value , I will start working on it

Related discussion:
text: show={solutionOpen} > <Modal.Header closeButton={true}> <del> <Modal.Title id='contained-modal-title'> </del> <ins> <Modal.Title id='contained-modal-title' className='text-center'> </ins> {`${username}'s Solution to ${ idToNameMap.get(id)?.challengeTitle ?? '' }`}
Code change:
Overly strict whitespace tests The space between the input element and closing label tag shouldn't matter, right? No response All No response
Here is the markdown file: we can use regex in these tests Something like , but we have to use regex which is headache.
could we just trim the string?
Sure, I don't mind, because they aren't required to type text in this step, but the description will be altered
I'm not sure I see why the description would be changed. I was thinking of this:
Yes, this line while be unnecessary. And the test will check for `EnterYourLastName:<input' But I maybe misunderstanding something
Trim won't remove spaces in the middle. We are comparing against:

Related discussion:
text: You should add the first `input` after the `label` text `Enter Your First Name:`, and include a space after the colon. ```js <del> assert.equal(document.querySelectorAll('label')?.[0]?.innerHTML, 'Enter Your First Name: <input>'); </del> <ins> assert.equal(document.querySelectorAll('label')?.[0]?.innerHTML.trim(), 'Enter Your First Name: <input>'); </ins> ``` You should add the second `input` after the `label` text `Enter Your Last Name:`, and include a space after the colon. ```js <del> assert.equal(document.querySelectorAll('label')?.[1]?.innerHTML, 'Enter Your Last Name: <input>'); </del> <ins> assert.equal(document.querySelectorAll('label')?.[1]?.innerHTML.trim(), 'Enter Your Last Name: <input>'); </ins> ``` You should add the third `input` after the `label` text `Enter Your Email:`, and include a space after the colon. ```js <del> assert.equal(document.querySelectorAll('label')?.[2]?.innerHTML, 'Enter Your Email: <input>'); </del> <ins> assert.equal(document.querySelectorAll('label')?.[2]?.innerHTML.trim(), 'Enter Your Email: <input>'); </ins> ``` You should add the fourth `input` after the `label` text `Create a New Password:`, and include a space after the colon. ```js <del> assert.equal(document.querySelectorAll('label')?.[3]?.innerHTML, 'Create a New Password: <input>'); </del> <ins> assert.equal(document.querySelectorAll('label')?.[3]?.innerHTML.trim(), 'Create a New Password: <input>'); </ins> ``` # --seed--
Code change:
there is a typo in cramers-rule challenge, is used instead of which is wrong these sentences should be changed to Here is the markdown file 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.
I wanted to work on this issue
feel free to open pull request after you have read

Related discussion:
text: # --description-- <del> <a href="https://rosettacode.org/wiki/Cramer%27s_rule" target="_blank" rel="noopener noreferrer nofollow">Cramer's rule</a> is a formula for solving a system of linear equations by using the determinants of matrixes formed from subsets of the coefficients and right hand side values. </del> <ins> <a href="https://rosettacode.org/wiki/Cramer%27s_rule" target="_blank" rel="noopener noreferrer nofollow">Cramer's rule</a> is a formula for solving a system of linear equations by using the determinants of matrices formed from subsets of the coefficients and right hand side values. </ins> <del> The determinate of a matrix with 2 rows and two columns is given by: </del> <ins> The determinant of a matrix with 2 rows and two columns is given by: </ins> $begin{aligned}|A|={begin{vmatrix}a&bc&dend{vmatrix}}=ad-bc.end{aligned}$
Code change:
In the page above, a question (Which activation function switches values between -1 and 1?) uses 'switches' when the actual word should be 'squishes' (like Tim says in the video) The question should be: Which activation function squishes values between -1 and 1? No response Device: Laptop OS: Windows 10 Browser: Chrome Version: 103.0.5060.114 No response
Just changing the word in this file should suffice, let me know if you want me to open a PR.
please go ahead.

Related discussion:
text: ## --text-- <del> Which activation function switches values between -1 and 1? </del> <ins> Which activation function squishes values between -1 and 1? </ins> ## --answers--
Code change:
There is a typo in learn-intermediate-css-by-building-a-picasso-painting, is used instead of . Here is the typo Here is the markdown file 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.
can i take this up
feel free to open pull request after you have read the guidelines

Related discussion:
text: assert(document.querySelectorAll('.characters > div')?.length === 4); ``` <del> Your two new `div` elemnts should have the `class` set to `blue`. </del> <ins> Your two new `div` elements should have the `class` set to `blue`. </ins> ```js const divs = document.querySelectorAll('.characters > div');
Code change:
In both the sentences on this line the word should be surrounded with backticks: In this line there is a period (full-stop) where there should be a comma. 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: 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.
Hello, I want to work on this.
Heyy, is this assigned to anyone?
I will work on this.
Hey! I would like to work on this issue.
Hi I have fixed the issue and you can find the PR here
Is this issue available or has it been resolved?

Related discussion:
text: assert(isValidSolution_(solve24(testCases_[1]))); ``` <del> `solve24("6789")` should return `(6*8)/(9-7)`. `(8*6)/(9-7)`, or a similar valid string </del> <ins> `solve24("6789")` should return `(6*8)/(9-7)`, `(8*6)/(9-7)`, or a similar valid string </ins> ```js assert(isValidSolution_(solve24(testCases_[2])));
Code change:
In both the sentences on this line the word should be surrounded with backticks: In this line there is a period (full-stop) where there should be a comma. 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: 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.
Hello, I want to work on this.
Heyy, is this assigned to anyone?
I will work on this.
Hey! I would like to work on this issue.
Hi I have fixed the issue and you can find the PR here
Is this issue available or has it been resolved?

Related discussion:
text: assert($('p > a').length); ``` <del> The link's href value should be `https://freecatphotoapp.com`. You have either omitted the href value or have a typo. </del> <ins> The link's `href` value should be `https://freecatphotoapp.com`. You have either omitted the `href` value or have a typo. </ins> ```js const nestedAnchor = $('p > a')[0];
Code change:
The word json should be capitalized in the hint text: The endpoint /json should serve the json object {"message": "Hello json"} Referenced challenge file: curriculum/challenges/english/05-back-end-development-and-apis/basic-node-and-express/serve-json-on-a-specific- Use capitals instead of lowercase letters when referring to JSON No response Not applicable No response

Related discussion:
text: # --hints-- <del> The endpoint `/json` should serve the json object `{"message": "Hello json"}` </del> <ins> The endpoint `/json` should serve the JSON object `{"message": "Hello json"}` </ins> ```js (getUserInput) =>
Code change:
In RTL, these texts will be broken, because they are a mix of special character, numbers, which follow different direction in RTL than LTR. ~1.2.13 1.2.x 1.x.x To fix these, add code elements around the version number. File: change to there are multiple instances in these files All of them should be wrapped in code elements Speaking of weird interactions. Using special characters that relay on direction will break in RTL, because it will be reverted, in example. will be we can wrap in
I want to contribute towards this issue. Can you assign this issue to me.
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

Related discussion:
text: # --instructions-- <del> Add version "1.1.0" of the `@freecodecamp/example` package to the `dependencies` field of your `package.json` file. </del> <ins> Add version `1.1.0` of the `@freecodecamp/example` package to the `dependencies` field of your `package.json` file. </ins> **Note:** `@freecodecamp/example` is a faux package used as a learning tool.