jiebi/CodeConvo huggingface dataset:
Change:
], [ "http://i.imgur.com/sfsidp6.jpg", <del> "El texto "Volverse bueno programa requiere tiempo."", </del> <ins> "El texto "Volverse bueno programando requiere tiempo."", </ins> "Volverse bueno programando requiere tiempo. No esperarías derrotar a un maestro del ajedrez después de jugar por 3 meses. No esperes crear el próximo Facebook después de programar 3 meses.", "" ],

Discussion:
[X] Learn how Free Code Camp Works [X] Create a GitHub Account and Join our Chat Rooms [X] Configure your Code Portfolio [X] Join a Campsite in Your City [X] Learn What to Do If You Get Stuck
Change:
```js assert.strictEqual( (function () { <del> bob.setFirstName('Haskell'); return bob.getFullName(); </del> <ins> _test_bob.setFirstName('Haskell'); return _test_bob.getFullName(); </ins> })(), 'Haskell Ross' );

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:
<Spacer size='medium' /> <h2>{t('learn.fill-in-the-blank')}</h2> <Spacer size='small' /> <del> <ObserveKeys> </del> <ins> {/* what we want to observe is ctrl/cmd + enter, but ObserveKeys is buggy and throws an error if it encounters a key combination, so we have to pass in the individual keys to observe */} <ObserveKeys only={['ctrl', 'cmd', 'enter']}> </ins> <div> <p> {splitSentence.map((s, i) => {

Discussion:
The English curriculum was as an upcoming change. A new "Fill in the blank" style challenge was created for it. When you go to one of these challenges and try to type an , it doesn't show up (sometimes). If you type it really quickly after a different character, it does show up. I'm quite sure the hotkeys are interfering - if you turn them off, the problem goes away. The is the one that we noticed, but there could be other characters that have this issue as well. Fill in the blank style challenges the codebase with the upcoming changes shown to one of the fill in the blank challenges: http://localhost:8000/learn/a2-english-for-developers/learn-greetings-in-your-first-day-at-the-office/right to type an in the blank, it doesn't work of keyboard shortcuts in settings, it does work All keys to work all the time. Related discussion:
Change:
"title": "Generate Random Whole Numbers within a Range", "difficulty":"9.9829", "description":[ <del> "We can use a certain mathematical expression to get a random number between between two numbers.", </del> <ins> "We can use a certain mathematical expression to get a random number between two numbers.", </ins> "<code>Math.floor(Math.random() * (max - min + 1)) + min</code>", "By using this we can control the output of a random number." ],

Discussion:
Challenge has an issue. "We can use a certain mathematical expression to get a random number between between two numbers." double between Best regards. misuta !
Change:
| passport-linkedin-oauth2        | Sign-in with LinkedIn plugin.                                        | | passport-oauth                  | Allows you to set up your own OAuth 1.0a and OAuth 2.0 strategies.   | | request                         | Simplified HTTP request library.                                     | <del> | lodash                          | Handy JavaScript utlities library.                                   | </del> <ins> | lodash                          | Handy JavaScript utilities library.                                   | </ins> | uglify-js                       | Dependency for connect-assets library to minify JS.                  | | mocha                           | Test framework.                                                      | | chai                            | BDD/TDD assertion library.                                           |

Discussion:
The updated topic banner on the is covering up a couple of the table headers here <img width="1087" alt="Screen Shot 2023-01-03 at 9 51 13 PM" src="https://user-" to for a new update topic banner to show up on homepage. the issue There should be some space between the banner and the table headers in the forum like it was before. <img width="1112" alt="Screen Shot 2023-01-03 at 9 47 33 PM" src="https://user-"- Device: [mac book pro] OS: [MacOS] Browser: [ Chrome] No response
div id="ember47" class="categories-and-latest ember-view" Try adding a padding-top to this element. I used around 100px
this issue may be with the discourse forum, itself. If it's in the discourse forum and it's repeated in different forums, not just freeCodeCamp, then it should be solved there. Removing the below properties fixed it in my devtools.
I looked at the following communities that use discourse and didn't notice the problem there. At first glance, it looks like it is only on the fcc forum page.
Change:
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:

Discussion:
! 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.
Change:
const { close, isOpen, <ins> isSignedIn, </ins> submitChallenge, handleKeypress, message,

Discussion:
I'm testing beta. Describe the bug When submitting a challenge using Ctrl+Enter it automatically completes also the following challenge. To Reproduce Steps to reproduce the behavior: a logged out user to the first challenge it Ctrl+Enter marks as solved also the following challenge and you can go ahead Expected behavior It should solve the current challenge and present the following one. Screenshots As you can see I can submit multiple challenges: ! Desktop (please complete the following information): OS: Ubuntu Browser Firefox Version 69.0.1 Additional context I've been able to reproduce it on but I could skip only one challenge. On my local dev I managed to skip 4.
I've digged a bit on this issue, and I think the problem is somehow related to this line: As this is triggered also when I'm submitting the finished challenge. If I comment that line out everything works fine, meaning I can Ctrl+Enter to run the tests and submit and the bug disappear. I'm going to dig a bit more, but does that makes any sense to you?
Thanks for testing our beta version and reporting this! You rock. /cc
Thanks It looks like I didn't test the logged out behaviour. so, what do we want to happen for logged out users? The issue I see is that it currently says 'Submit and go to the next challenge', but clearly they can't submit. For now, do we just want them to see the same as logged in users (and quietly not submit)?
Here is what I would do: ! Of course the modal should close as well like you mentioned. The signin button can link to same as the home page CTA. And we could remove from the first button
Turns out it's not working properly even when the user is logged in. I think it's a race condition between and . was quite right, the problem is on that line. It shouldn't be called if the modal is open since it's listening to as well.
yes I think it's with the in the editor and the in the . I can give a further look later if you wish.
Thanks for the offer but I think (with your help!) I've fixed it. It seemed only fair, since I introduced the bug.
Excellent!
Release on its way, should be available within the hour.
Change:
import PropTypes from 'prop-types'; import { reduxForm } from 'redux-form'; <del> import { FormFields, BlockSaveButton, BlockSaveWrapper } from './'; </del> <ins> import { FormFields, BlockSaveButton, BlockSaveWrapper, formatUrlValues } from './'; </ins> const propTypes = { buttonText: PropTypes.string,

Discussion:
Describe the bug Submitting the solution link as "" produces an error. Submitting "" does not. Whenever we submit a solution link "" it is appended with "/api/" so resultant solution gives "Not Found/Timeout error" and the same link submitted without "/" like "" gives solution as accepted. To Reproduce Steps to reproduce the behavior: to on 'solution input box' "https://freecodecamp-" solution gets accepted. "https://freecodecamp-" error Expected behavior This should be tackled as Chrome appends "/" by default to any domain URL you copy, so people who would copy URL from search bars will get URLs appended with "/" hence producing wrong results. This file shows the testString is directly appended with "/api" without checking if the input URL contains "/" at the end or not. Screenshots ! Desktop (please complete the following information): OS: Ubuntu 18 Browser: Google Chrome Version 73.0.3683.75
I am not having this problem - my project will pass the tests with or without the trailing https://fcc-manage-npm- and https://fcc-manage-npm- both pass for me in chrome Can you share a share a screen shot of the error?
Updated
hmm, it's strange that my project will give a good response when that string is appended but not yours https://fcc-manage-npm- - package as response https://freecodecamp- - not found as response not sure what is causing this - the only difference seems to be that I'm using glitch and you're using heroku in fact mine will work like this... https://fcc-manage-npm- so maybe how the two websites process a route or something? can they do that? glitch strips out the extra or something? weird Either way - I think a solution like you suggested would probably work - check if the submitted URL ends in a and remove it if it's there. This same problem probably occurs throughout the whole backend area all over the place.
Weird, would u check with an alternative hosting if the problem persists? If the problem persists the last solution would be stripping the las from input url
Did you try this Above is the actual link processed in the test with an appended , It gives a response with the same as it is shown in the screenshot I posted. So I guess the problem is with the routing of Heroku.
Change:
font-size: 1.5rem; } <ins> .donation-modal .btn-link:focus { outline-width: 1px; outline-style: solid; } </ins> @media screen and (max-width: 991px) { .donation-icon-container { margin: 30px;

Discussion:
Describe the bug The popup modal for becoming a sponsor cannot be closed by just using the keyboard. To Reproduce Steps to reproduce the behavior: some challenges which triggers the modal Expected behavior The modal closes when pressing the Esc key The modal closes when clicking somewhere outside of the modal I can navigate to the "Ask me later" button with the tab key to close the modal Screenshots ! Desktop (please complete the following information): OS: Ubuntu Browser: chrome and firefox Version: Latest Smartphone (please complete the following information): Unknown
This is by design to the best of my knowledge. I would defer to Quincy for a confirmation.
Thanks for reporting that the "ask me later" link should be accessible by tab but you're correct - it isn't. We will fix this. We decided not to have escape key / clicking outside of the modal to dismiss this modal (like we do with other modals) since the goal is to get people to actually slow down and consider donating to our nonprofit - not to just mash the escape key.
Change:
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');

Discussion:
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
Change:
} ::selection { <del> background-color: rgba(var(--highlight-color), 0.3); </del> <ins> background-color: var(--selection-color); </ins> } @media (max-width: 500px) {

Discussion:
If I try to select text with my mouse so I can copy/paste it or something - the selected text has the same background as the rest of the page. Making it difficult to see what is selected, and frustrating to try and select, well anything. ! This seems to occur across the entire site. Perhaps it was intentional or something, but I don't think it's a good ux. Note: the text does actually get selected, so you can still copy/paste it. Tested on Chrome and Firefox
Good catch. It'll be this PR: which didn't scope the change to challenge descriptions. I'll put together a quick fix. Edit: nevermind, it's just that has to be passed four values, not two. I don't quite understand how this ever worked.
Change:
You are provided sentences with some missing words, like nouns, verbs, adjectives and adverbs. You then fill in the missing pieces with words of your choice in a way that the completed sentence makes sense. <del> Consider this sentence - It was really **____**, and we **____** ourselves **____**. This sentence has three missing pieces- an adjective, a verb and an adverb, and we can add words of our choice to complete it. We can then assign the completed sentence to a variable as follows: </del> <ins> Consider this sentence: ```md It was really ____, and we ____ ourselves ____. ``` This sentence has three missing pieces- an adjective, a verb and an adverb, and we can add words of our choice to complete it. We can then assign the completed sentence to a variable as follows: </ins> ```js const sentence = "It was really " + "hot" + ", and we " + "laughed" + " ourselves " + "silly" + ".";

Discussion:
The following sentence in the Word Blanks lesson should have a code block (three backticks) surrounding the code section which is directly referring to the completed code block below it on the page. Instead of: Consider this sentence - It was really , and we ourselves . It should say: Consider this sentence - The sentence has been translated because of the missing code blocks (and will not make sense as it's a direct reference to the actual code shown below it) The sentence with the blanks should be in a code block that does not wrap. (three backticks above and below it) Instead of: Consider this sentence - It was really , and we  ourselves . It becomes: Consider this sentence - No response n/a No response
Question for those in the know. Do we have a way to keep content from being translated that doesn't involve using the triple back tick code block? I would be cool to use sr-only text "Blank" (or something similar) for those underlines to represent missing text. But we can't do that using the code blocks since they are wrapped in a .
Do they get wrapped in a if they're inline code?
It looks like they are just wrapped in strong tags in crowdin at least. I don't see a pre tag.
pre sounds like a plausible idea. We should go ahead with that.
The challenge require the sentence to not be translated, and element don't stop translation, it gives the choice to the translation to translate or not, which isn't ideal. Here is the file, to solve this issue, change the structure of the sentence to md It was really , and we ourselves . ` 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 prolific contributors will guide you through this.
nope, but we can open an issue for this yes
Can you assign me this issue
Only issues with the or 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:
button1.onclick = buyHealth; button2.onclick = buyWeapon; button3.onclick = goTown; } --fcc-editable-region--

Discussion:
In step 41 of the RPG project in the JS beta curriculum, the instruction states: Change the property of the to be "You enter the store.". I believe this sentence is not 100% clear without either removing the word “the” in front of or adding the word “variable” to the right of . As it is, the sentence misleads a bit because it says “the ” which makes it sound like it is referring to some text somewhere, instead of the text variable. so either: Change the property of to be "You enter the store.". or Change the property of the variable to be "You enter the store.". will be clearer. Please look at questions posted on the forum for examples of people missing the point of the request with the current wording. I personally prefer the second one. N/A N/A N/A Example of confused learner:
Change:
"Vamos a darle un id único a cada uno de nuestros elementos <code>div</code> que tienen la clase <code>well</code>.", "Recuerda que puedes darle a un elemento un id como el siguiente:", "<code>&#60;div class="well" id="center-well"&#62;</code>", <del> "Dale al pozo de la izquireda el id <code>left-well</code>. Al pozo de la derecha, dale un <code>id</code> de <code>right-well</code>." </del> <ins> "Dale al pozo de la izquireda el id <code>left-well</code>. Al pozo de la derecha, dale un id <code>right-well</code>." </ins> ] }, {

Discussion:
[ ] Use Responsive Design with Bootstrap Fluid Containers [ ] Make Images Mobile Responsive [ ] Center Text with Bootstrap [ ] Create a Bootstrap Button [ ] Create a Block Element Bootstrap Button [ ] Taste the Bootstrap Button Color Rainbow [ ] Call out Optional Actions with Button Info [ ] Warn your Users of a Dangerous Action [ ] Use the Bootstrap Grid to Put Elements Side By Side [ ] Ditch Custom CSS for Bootstrap [ ] Use Spans for Inline Elements [ ] Create a Custom Heading [ ] Add Font Awesome Icons to our Buttons [ ] Add Font Awesome Icons to all of our Buttons [ ] Responsively Style Radio Buttons [ ] Responsively Style Checkboxes [ ] Style Text Inputs as Form Controls [ ] Line up Form Elements Responsively with Bootstrap [ ] Create a Bootstrap Headline [ ] House our page within a Bootstrap Container Fluid Div [ ] Create a Bootstrap Row [ ] Split your Bootstrap Row [ ] Create Bootstrap Wells [ ] Add Elements within your Bootstrap Wells [ ] Apply the Default Bootstrap Button Style [ ] Create a Class to Target with jQuery Selectors [ ] Add ID Attributes to Bootstrap Elements [ ] Label Bootstrap Wells [ ] Give Each Element a Unique ID [ ] Label Bootstrap Buttons [ ] Use Comments to Clarify Code
Working on this one
Change:
"assert.deepEqual(inventory([], [[2, 'Hair Pin'], [3, 'Half-Eaten Apple'], [67, 'Bowling Ball'], [7, 'Toothpaste']]), [[2, 'Hair Pin'], [3, 'Half-Eaten Apple'], [67, 'Bowling Ball'], [7, 'Toothpaste']]);", "assert.deepEqual(inventory([[0, 'Bowling Ball'], [0, 'Dirty Sock'], [0, 'Hair pin'], [0, 'Microphone']], [[1, 'Hair Pin'], [1, 'Half-Eaten Apple'], [1, 'Bowling Ball'], [1, 'Toothpaste']]), [[1, 'Bowling Ball'], [1, 'Dirty Sock'], [1, 'Hair pin'], [1, 'Half-Eaten Apple'], [1, 'Microphone'], [1, 'Toothpaste']]);" ] <ins> }, { "_id": "a2f1d72d9b908d0bd72bb9f6", "name": "Make a Person", "difficulty": "3.12", "description": [ "Fill in the object constructor with the methods specified in the tests.", "Those methods are getFirstName(), getLastName(), getFullName(), setFirstName(), setLastName(), and setFullName().", "These methods must be the only available means for interacting with the object.", "There will be some linting errors on the tests, you may safely ignore them. You should see undefined in the console output." ], "challengeEntryPoint": "var bob = new Person('Bob Ross');", "challengeSeed": "var Person = function(firstAndLast) {n    return firstAndLast;rn};", "tests": [ "expect(Object.keys(bob).length).to.eql(6);", "expect(bob instanceof Person).to.be.true;", "expect(bob.firstName).to.be.undefined();", "expect(bob.lastName).to.be.undefined();", "expect(bob.getFirstName()).to.eql('Bob');", "expect(bob.getLastName()).to.eql('Ross');", "expect(bob.getFullName()).to.eql('Bob Ross');", "bob.setFirstName('Happy');", "expect(bob.getFirstName()).to.eql('Happy');", "bob.setLastName('Trees');", "expect(bob.getLastName()).to.eql('Trees');", "bob.setFullName('George Carlin');", "expect(bob.getFullName()).to.eql('George Carlin');" ] </ins> } ]

Discussion:
<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --In the challenge CSS Flexbox: Use the flex-wrap Property to Wrap a Row or Column, the value wrap-reverse is defined: wrap-reverse: "wraps items from right-to-left if they are in a row, or bottom-to-top if they are in a column." This definition is not correct. Actually wrap-reverse wrap content changing de cross-axis direction, bottom to top in a row, and left to right in a column. Browser Name: Browser Version: Operating System:
I'd like to take on this issue please
Since the curriculum content is in the repo, I submitted the PR there.
Ok, thanks Simon.
Change:
## --video-solution-- <del> 1 </del> <ins> 2 </ins>

Discussion:
The answer is wrong for this page. N/A I expected the correct answer to be 'They focus on presenting information rather than programming logic.' But the correct answer appears to be 'They lack the ability to create webpages.'. <img width="845" alt="image" src=""I think the system is irrelevant, so I am skipping this. No response