jiebi/CodeConvo huggingface dataset:
Change:
```js const stat = document.querySelectorAll('.stat')[2]; <del> assert.equal(stat.innerText, 'Gold: 50'); </del> <ins> assert.equal(stat.innerText.trimEnd(), 'Gold: 50'); </ins> ``` # --seed--

Discussion:
I was helping a with in the new JS curriculum and they weren't able to pass the tests because of the way they formatted their answer. Here was their code We should allow for that code to pass because they got it right. Here are the current tests For all of the tests that use we should instead update it to use a regex solution with instead. That will make it possible for campers to submit answers with different spacing. All of the tests with the to make sure they have a or element should stay the same though. For those, interested in working on this issue, please read through the first. There will be no assignment for this issue. If you are interested in working on it, then you can submit a PR The following code should pass regardless of different formatting with spaces. No response Device: [e.g. iPhone 6, Laptop] OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04] Browser: [e.g. Chrome, Safari] Version: [e.g. 22] No response
Change:
| Feature | Variable Type | Variable      | Value Type | |:-------:|:------------:|:-------------:|:----------:| <del> | Age | Objective Feature | age | int (days) | | Height | Objective Feature | height | int (cm) | | Weight | Objective Feature | weight | float (kg) | | Gender | Objective Feature | gender | categorical code | | Systolic blood pressure | Examination Feature | ap_hi | int | | Diastolic blood pressure | Examination Feature | ap_lo | int | | Cholesterol | Examination Feature | cholesterol | 1: normal, 2: above normal, 3: well above normal | | Glucose | Examination Feature | gluc | 1: normal, 2: above normal, 3: well above normal | | Smoking | Subjective Feature | smoke | binary | | Alcohol intake | Subjective Feature | alco | binary | | Physical activity | Subjective Feature | active | binary | | Presence or absence of cardiovascular disease | Target Variable | cardio | binary | </del> <ins> | Age | Objective Feature | `age` | int (days) | | Height | Objective Feature | `height` | int (cm) | | Weight | Objective Feature | `weight` | float (kg) | | Gender | Objective Feature | `gender` | categorical code | | Systolic blood pressure | Examination Feature | `ap_hi` | int | | Diastolic blood pressure | Examination Feature | `ap_lo` | int | | Cholesterol | Examination Feature | `cholesterol` | 1: normal, 2: above normal, 3: well above normal | | Glucose | Examination Feature | `gluc` | 1: normal, 2: above normal, 3: well above normal | | Smoking | Subjective Feature | `smoke` | binary | | Alcohol intake | Subjective Feature | `alco` | binary | | Physical activity | Subjective Feature | `active` | binary | | Presence or absence of cardiovascular disease | Target Variable | `cardio` | binary | </ins> ## Tasks

Discussion:
This markdown table is missing backticks around the variables (third column). 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.
Hi , this seems like an issue that i can handle. I am new to freecodecamp repository and would love to contribute
Thank you for your interest! I suggest looking at issues that do not have linked PRs already
Change:
"tests": [], "type": "basejump", "challengeType": 4, <del> "isRequired": true </del> <ins> "isRequired": true, "nameEs": "Microservicio para acortar URLs", "descriptionEs": [ "<strong>Objetivo:</strong> Arma una aplicación de Pila Completa en JavaScript que sea funcionalmente similar a esta: <a href='https://shurli.herokuapp.com/' target='_blank'>https://shurli.herokuapp.com/</a> y despliegala en Heroku.", "Ten en cuenta que para cada proyecto, deberías crear un nuevo repositorio en GitHub y un nuevo proyecto en Heroku. Si no recuerdas como hacer esto, vuelve a visitar <a href='/challenges/get-set-for-our-back-end-development-projects'>http://freecodecamp.com//challenges/get-set-for-our-back-end-development-projects</a>.", "Aquí están las historias de usuario específicas que debes implementar para este proyecto:", "<strong>Historia de Usuario:</strong> Puedo pasar una URL como parámetro y recibiré una URL acortada en la respuesta JSON.", "<strong>Historia de Usuario:</strong> Si paso una URL inválido que no siga el formato válido http://www.exmaple.com, en cambio la respuesta JSON contendrá un mensaje de error.", "<strong>Historia de Usuario:</strong> Cuando visite el URL acortado, me redirigirá a mi enlace original.", "Una vez que hayas terminado de implementar estas historias de usuarios, pulsa el botón "I've completed this challenge" e introduce los URLs de tu repositorio en GitHub y de tu aplicación en vivo en Heroku.", "Puedes obtener retroalimentación sobre tu proyecto por parte de otros campistas, compartiendolo en nuestra <a href='//gitter.im/freecodecamp/codereview' target='_blank'>Sala de chat para revisión de código</a>. También puedes compartirlo en Twitter y en el campamento de tu ciudad (en Facebook)." ] </ins> }, { "id": "bd7158d8c443edefaeb5bdee",

Discussion:
[x] Get Set for our Back End Development Projects [x] Timestamp Microservice [x] Request Header Parser Microservice [x] URL Shortener Microservice [x] Image Search Abstraction Layer [x] File Metadata Microservice
Currently working on this.
I just used part of the PR that you closed. Thank you. Blessings.
Change:
text-align: center; } <ins> .big-ion-up-arrow  #upvote, #reply-to-main-post { cursor: pointer; } </ins> .story-up-votes { padding-top: 0px; margin-left: -5px;

Discussion:
In the challenge "Debugging: Understanding the Differences between the freeCodeCamp and Browser Console" the objective asks the user to whereas in the content, the user has access to two prewritten variables: and , no variable exists. Browser Name: Google Chrome Browser Version: 67.0.3396.99 (64 bits) Operating System: Windows 10 !
I'd like to give it a go.
Change:
1. Guía de estilo sobre cómo crear Guías. 2. Guía de estilo sobre cómo crear Desafíos de programación. <del> ## Referencía rápida de comandos para trabajar en local </del> <ins> ## Referencia rápida de comandos para trabajar en local </ins> Una referencia rápida de los comandos que necesitarás cuando trabajes en local.

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: There's a spelling mistake on the Spanish documentation's ReadMe Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Chrome Browser Version: 69.0.3497.100 Operating System: MacOS If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box): !
Change:
35, 0.35 ), <del> 75300 </del> <ins> 75900 </ins> ); ```

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:
More resources: <del> -  <a href="https://github.com/ine-rmotr-curriculum/FreeCodeCamp-Pandas-Real-Life-Example"target="_blank" r el="noopener noreferrer nofollow">Notebooks on GitHub</a> </del> <ins> -  <a href="https://github.com/ine-rmotr-curriculum/FreeCodeCamp-Pandas-Real-Life-Example" target="_blank" rel="noopener noreferrer nofollow">Notebooks on GitHub</a> </ins> -  <a href="https://colab.research.google.com/github/googlecolab/colabtools/blob/master/notebooks/colab-github-demo.ipynb" target="_blank" rel="noopener noreferrer nofollow">How to open Notebooks from GitHub using Google Colab.</a> # --question--

Discussion:
Out of the 2 points in the "More resources:" section, the first point has a typo that has caused it to render in an unintended manner. Check this the point should be rendered as "Notebooks on GitHub" with an underline and should be clickable. ! I used a web browser(chrome) but I believe its a common issue for every device. No response
Change:
<Link className='large-p underlined-link' external={true} to='/donate'> make a tax-deductible donation here </Link> <ins> . </ins> </p> <del> .<Spacer /> </del> <ins> <Spacer /> </ins> <BigCallToAction /> <Spacer /> <Spacer />

Discussion:
There's a period on its own paragraph on the home page... ! Here's where the issue lies... Perhaps adding the period in the tag
Change:
_ = require('lodash'), fs = require('fs'), <del> constantStrings = require('constantStrings.json'), </del> <ins> constantStrings = require('./constantStrings.json'), </ins> User = require('../models/User'), Challenge = require('./../models/Challenge'), Story = require('./../models/Story'),

Discussion:
Seeing this in production here: solution: Use and/or check for user existence and end early. cc:
Change:
] ], "type": "hike", <del> "challengeType": 6 </del> <ins> "challengeType": 6, "nameEs": "JavaScript Lingo: Encontrando e indexando valores en vectores", "descriptionEs": [ "Hay muchas razones por las que puedes necesitar acceder a una determinada pieza de información al interior de un conjunto más amplio, lo cual haces referenciando su índice.", "No vamos a entrar en la sintaxis, pero debes saber que el primer elemento en un vector tiene en realidad el índice 0.", "Esto también aplica para cadenas y objetos. Todos estos índices comienzan en 0, por lo que si en tu programa usas <code>miVector[2]</code>, realmente vas a obtener la tercera pieza de información del vector <code>miVector</code>." ] </ins> }, { "id": "56b15f15632298c12f315188",

Discussion:
[ ] JavaScript Lingo: MDN and Documentation [ ] JavaScript Lingo: Value Types [ ] JavaScript Lingo: Variables & camelCase [ ] JavaScript Lingo: Arrays & Objects [ ] JavaScript Lingo: Finding and Indexing Data in Arays [ ] JavaScript Lingo: Manipulating Data [ ] JavaScript Lingo: Math [ ] JavaScript Lingo: Loops [ ] JavaScript Lingo: Regular Expressions
Working on this one
Change:
return challenge.id === challengeMeta.id; }); <del> createFiles(savedChallenge?.challengeFiles || challengeFiles || []); </del> <ins> createFiles( mergeChallengeFiles(challengeFiles, savedChallenge?.challengeFiles) ); </ins> initTests(tests); if (showProjectPreview) openModal('projectPreview');

Discussion:
Code in the doesn't seem to be evaluated in production for Cash Register project. (Edit)This appear to be related to the code saving / resetting. In development there isn't such issue. Including building production client and serving it. This is actually the only challenge from the new challenges that is using . Old challenges are working fine when using . Both when names start with ` might be somehow deciding factor, but there isn't any other new challenge to check that. After adding function to user code, error moves normally to the next used in test non-defined variable - and then . Steps to reproduce the behavior: (Be logged in, and have previously saved code for project). to solution. Example: tests. errors. Code present in the should be evaluated after the user code, and before running tests. (Edit) When logged out there's different error, or no errors at all(?!). This might be somehow related to saving / resetting code.
Thanks again for the detailed report, You were dead right about the issue: it's due to the fact that saved challenges don't include head or tail (aka and ) and we were just using them in place of challengeFiles (which does have that stuff).
I have marked this as a showstopper seeing as it is affecting camper's ability to complete a certificate project. It would be nice to get the fix in as soon as possible, so the forum isn't filled with support threads for this.
Change:
id: 5a94fe8569fb03452672e464 title: Create Grids within Grids challengeType: 0 <del> videoUrl: 'https://scrimba.com/p/pByETK/c6N78Ap' </del> forumTopicId: 301128 ---

Discussion:
Challenge: Video: The code is supposed to create a two-column layout but there is a typo. Video has: Should be: It would be nice to have the video fixed if possible.
I would like to help but I need some more information like the download link for the video to edit it and overwrite columns on colums.
Is it possible to get this video updated?
this still needs to be fixed, if possible.
Change:
# --description-- <del> Notice that your marker doesn't seem to have any color. The background color was actually applied, but since the marker `div` element is empty, it doesn't have any width or height by default. </del> <ins> Notice that your marker doesn't seem to have any color. The background color was actually applied, but since the marker `div` element is empty, it doesn't have any height by default. </ins> In your `.marker` CSS rule, set the `width` property to `200px` and the `height` property to `25px`.

Discussion:
In the "Learn CSS Colors by Building a Set of Colored Markers" project, the step 12 description reads: Notice that your marker doesn't seem to have any color. The background color was actually applied, but since the marker div element is empty, it doesn't have any width or height by default. The div doesn't have any height by default, but as it's a block-level element it spans the entire width of the containing element (body). N/A N/A No response N/A No response
As not a native english speaker, I am confused. Can you provide more details, and an alternative you think will fit the current step?
I agree, it should only be height, not width. As you pointed out the element does have a width.
Suggested change, remove the "width or" from the sentence.
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: Change to make: the . Explanation: the div being a block element spans the width of the entire page, as such it is wrong information that should be removed. It is correct to say that it doesn't have an heigth. Please make sure you read , we prioritize contributors following the instructions in our guides. Join us in 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.
This is my first "real" contribution to open source (let alone to FCC), so although it is a pretty insignificant one, feel free to provide me with any feedback and/or constructive criticism please!
Change:
$(document).ready(function() { <ins> if (bonfireName !== undefined) { ga('send', 'event',  'Bonfire', 'load', bonfireName + ':' + Math.floor(Date.now() / 1000)); } </ins> var CSRF_HEADER = 'X-CSRF-Token';

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:
# --hints-- <del> You should have an `img` element with an `src` value of `https://cdn.freecodecamp.org/curriculum/cat-photo-app/relaxing-cat.jpg`. You may have accidentally deleted it. </del> <ins> You should have an `img` element with a `src` value of `https://cdn.freecodecamp.org/curriculum/cat-photo-app/relaxing-cat.jpg`. You may have accidentally deleted it. </ins> ```js assert(

Discussion:
I noticed that we are using both "a attribute" and "an attribute" in challenge descriptions. And we even use both in the same challenge (): <img width="596" alt="Screenshot 2024-05-21 at 22 44 29" src=""I think both are acceptable, since the articles depend on how we pronounce "src" ("es-ar-see" vs "source"), but I think we should stick to one for consistency. I'll let native English speakers make the call
I always pronounce it as , so I vote
I'm with Jessica on this one. The second sentence should be changed to .
This issue is open to first-time code contributors to this repository. The files that you should be updating are: Please make sure you read our as well as 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.
Hi everyone. This is a great pick up! I am happy to help out there and will get a initial PR up for review today :)
I have reviewed the changes made in PR, all the related file have been fixed correctly and checked locally.
Hello, This is my first Open source contribution so if you guys can give me feedback on what i did wrong and how i should correct it please guide me. from the seven files mentioned above, I have created pull request for each one which is as below. sorry if i messed up, as it is my first try. I am actively looking for any advice, suggestion, criticism. Thank You.
Change:
"id": "bad87fee1348bd9aedf08802", "title": "Uncomment HTML", "description": [ <del> "Commenting is a way that you can leave comments for other developers within your code without affecting the resulting output that is displayed the the end user.", </del> <ins> "Commenting is a way that you can leave comments for other developers within your code without affecting the resulting output that is displayed the end user.", </ins> "Commenting is also a convenient way to make code inactive without having to delete it entirely.", "Comments in HTML starts with <code>&#60;!--</code>, and ends with a <code>--&#62;</code>", "<hr>",

Discussion:
Typo in the challenge description !
i would like to do it :)
i would like to work on it
so would i.. :-)
Would love to work on this
Change:
As a reminder, this project is being built upon the following starter project on <a href='https://glitch.com/edit/#!/remix/clone-from-repo?REPO_URL=https://github.com/freeCodeCamp/boilerplate-advancednode/'>Glitch</a>, or cloned from <a href='https://github.com/freeCodeCamp/boilerplate-advancednode/'>GitHub</a>. A strategy is a way of authenticating a user. You can use a strategy for allowing users to authenticate based on locally saved information (if you have them register first) or from a variety of providers such as Google or GitHub. For this project we will set up a local strategy. To see a list of the 100's of strategies, visit Passports site <a href='http://passportjs.org/'>here</a>. Add <em>passport-local</em> as a dependency and add it to your server as follows: <code>const LocalStrategy = require('passport-local');</code> <del> Now you will have to tell passport to <b>use</b> an instantiated LocalStartegy object with a few settings defined. Make sure this as well as everything from this point on is encapsulated in the database connection since it relies on it! </del> <ins> Now you will have to tell passport to <b>use</b> an instantiated LocalStrategy object with a few settings defined. Make sure this as well as everything from this point on is encapsulated in the database connection since it relies on it! </ins> ```js passport.use(new LocalStrategy(

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: You said 'LocalStartegy' instead of 'LocalStrategy' Add a Link to the page with the problem: ! Tell us about your browser and operating system: Browser Name: Browser Version: Operating System: If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):
I'll take it up.
Change:
--primary-background: var(--gray-00); --highlight-color: var(--blue-dark); --highlight-background: var(--blue-light); <ins> --selection-color: var(--blue-dark-translucent); </ins> --success-color: var(--green-dark); --success-background: var(--green-light); --danger-color: var(--red-dark);

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.