jiebi/CodeConvo huggingface dataset:
Change:
In computer science, a <dfn>stack</dfn> is a data structure where items are stored in a <dfn>LIFO</dfn> (last-in-first-out) manner. If you imagine a stack of books, the last book you add to the stack is the first book you can take off the stack. Or an array where you can only `.push()` and `.pop()` elements. <del> The <dfn>call stack</dfn> is a collection of function calls stored in a stack structure. When you call a function, it is added to the top or of the stack, and when it returns, it is removed from the top / end of the stack. </del> <ins> The <dfn>call stack</dfn> is a collection of function calls stored in a stack structure. When you call a function, it is added to the top of the stack, and when it returns, it is removed from the top / end of the stack. </ins> You'll see this in action by creating mock call stack.

Discussion:
In the second paragraph of Step 47: Per my understanding of the text, the "or" (italic and bold) should be removed. There is nothing wrong with the testing for this step; it is just a typo. The "or" as pointed out above should be deleted. 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
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:
.site-footer .col-header, .trending-guides .col-header { text-align: center; <ins> font-family: var(--font-family-sans-serif); </ins> font-weight: 700; font-size: 16px; padding: 0 15px 15px;

Discussion:
The font family for the 'trending guides' and 'mobile app' titles in the footer should be 'Lato', sans-serif and the .footer-desc-col in the footer should be aligned with the trending find the related selector for the footer-desc-col and add the following style align-self: start; Also, find the related selector to the footer titles and add the following style: font-family: var(--font-family-sans-serif); <img width="679" alt="Screenshot 2024-02-07 at 3 26 07 PM" src=""
Hi, I'd like to take this issue please
We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labelled with or are open for contributions. Please make sure you read . We prioritize contributors following the instructions in our guide. Join us in or if you need help contributing - our community will be happy to assist you.
Change:
} else { assert.equal(winnerMsgElement.innerText, "Computer has won the game!"); } ``` You should hide the `optionsContainer` and if the player or computer has reached three points.

Discussion:
In the last part of step 5 in the rock, paper, scissors review project, the instructions given say: If there is a winner, you will want to show the resetGameBtn button and hide the optionsContainer so the player can play again. Tips You can use the property to show the resetGameBtn button and hide the optionsContainer. I have 2 concerns here. 1- at no point is there any mention of what type of display the resetGameBtn requires (is it block, inline-block, flex...?) I guessed 'block' which worked but the lack of this detail made me hesitate. 2- the mention of '' is confusing. In the previous project, using style.display was discussed before in the rpg project () but it was never referred to or mentioned as '' before. I wasn't sure if 'el' meant the element in question or if there was an actual 'el' variable somewhere. I ended up going with what made sense (replacing the 'el' with the element reference) but I'm not sure how easy this would be for someone with only the last few js projects under their belt. A clarification would be great here. N/A N/A No response N/A No response
I fixed the wording of the challenge to make it more clear to the user.
This issue has not been triaged yet, in the future please do not create a PR until then. Anyway, I will open it up to contribution.
my bad sir, won't happen again.
Change:
# --description-- <del> Because `elemValue` returns a function, your `addChars` function ultimately returns an array of function references. You want the `.map()` method to run the inner function of your `elemValue` function, which means you need to call `elemValue` instead of reference it. Pass `num` as the argument to your `elemValue` function. </del> <ins> Because `elemValue` returns a function, your `addCharacters` function ultimately returns an array of function references. You want the `.map()` method to run the inner function of your `elemValue` function, which means you need to call `elemValue` instead of reference it. Pass `num` as the argument to your `elemValue` function. </ins> <!-- TODO: Explain further? -->

Discussion:
The function used in the code, named : Instructions mention function: "Because returns a function, your function ultimately returns an array of function references..." Instruction should mention function, not function. No response Device: Laptop OS: Windows 10 Browser: Chrome 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.
can you assign me this issue?
We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labelled with or are open for contributions. Please make sure you read . We prioritize contributors following the instructions in our guide. Join us in or if you need help contributing - our community will be happy to assist you.
Change:
], 'id' ); <del> return updatedExisting; </del> <ins> return { updated: updatedExisting, isNewCompletionCount: updatedExisting.length - completedChallenges.length }; </ins> } function isTheSame(val1, val2) {

Discussion:
Completing projects from the settings page, submitting a url, dose not increment the points a user has. We need to push a new timestamp to the users array.
Hi I can help out with this. Am I correct in thinking that this should be done in the file? If so, I'll look at for some guidance.
Change:
--yellow-light: #ffc300; --yellow-dark: #4d3800; --blue-light: rgb(153, 201, 255); <ins> --blue-light-translucent: rgb(153, 201, 255, 0.3); </ins> --blue-dark: rgb(0, 46, 173); <ins> --blue-dark-translucent: rgb(0, 46, 173, 0.3); </ins> --green-light: #acd157; --blue-mid: #198eee; --purple-mid: darkviolet;

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:
# --hints-- <del> `Object.keys(bob).length` should return 6. </del> <ins> No properties should be added. `Object.keys(bob).length` should always return 6. </ins> ```js <del> assert.deepEqual(Object.keys(bob).length, 6); </del> <ins> assert.strictEqual( Object.keys((function () { let bob = new Person('Bob Ross'); bob.setFirstName('Haskell'); bob.setLastName('Curry'); bob.setFullName('John Smith'); return bob; })()).length, 6 ); </ins> ``` `bob instanceof Person` should return `true`.

Discussion:
If any of the methods create new properties on the object the two tests for the properties and the one for the keys should fail. Not sure if the intent of the challenge needs to be clarified a bit better. We might want to refer back to the previous challenge "Use Closure to Protect Properties Within an Object from Being Modified Externally". Challenge: Forum post:
this probably can be tested only if we add a call to the setters (maybe also the getters to be throughful) before testing for the number of properties it wouldn't be too difficult to do, if we arrive to the conclusion that it is needed
Hi i would like to contribute to this issue. can you please define problem statement.
An issue with the help wanted or first timers only label is open for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors. If you would like to contribute, and have not read the contributors docs, please do so here: If you have any issues with contributing, be sure to join us on the , or on the
Change:
<td><a href="/docs/arabic/CONTRIBUTING.md"> عربي </a></td> <td><a href="/docs/chinese/CONTRIBUTING.md"> 中文 </a></td> <td><a href="/docs/portuguese/CONTRIBUTING.md"> Português </a></td> <del> <td><a href="/docs/russian/CONTRIBUTING.md"> русский </a></td> </del> <ins> <td><a href="/docs/russian/CONTRIBUTING.md"> Русский </a></td> </ins> <td><a href="/docs/spanish/CONTRIBUTING.md"> Español </a></td> <ins> <td><a href="/docs/greek/CONTRIBUTING.md"> Ελληνικά </a></td> </ins> </tr> </table>

Discussion:
Describe your problem and how to reproduce it: When you try to change contibution guidelines language - you'll see how the languages table's order is desynchronized and in several languages you cannot find new Greek language. Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Chrome Browser Version: Version 74.0.3729.169 (Official Build) (64-bit) Operating System: Windows 10 Pro 1809 If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box): !
Change:
# --description-- <del> Add a `title` and a `meta` elements to the `head`. Give your project a title of `Registration Form`, and give a `charset` attribute with a value of `UTF-8` to your `meta` element. </del> <ins> Add a `title` and a `meta` element to the `head`. Give your project a title of `Registration Form`, and give a `charset` attribute with a value of `UTF-8` to your `meta` element. </ins> # --hints--

Discussion:
The sentence is worded strangley: We should refphrase this sentence to something like: ! 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
An issue with the first timers only label is open for contribution by first time contributors only. 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 from to
Can i work in this issue ?
Go for it my friend! Just make sure to create a new branch when you make the changes!
Change:
return value => (value ? fn(value) : value); } <ins> export function formatUrlValues(values, options) { return Object.keys(values).reduce((result, key) => { let value = values[key]; if (options.types[key] === 'url') { value = normalizeUrl(value, normalizeOptions); } return { ...result, [key]: value }; }, {}); } </ins> // formatUrl(url: String) => String export function formatUrl(url) { if (typeof url === 'string' && url.length > 4 && url.indexOf('.') !== -1) {

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:
--primary-background: var(--gray-90); --highlight-color: var(--blue-light); --highlight-background: var(--blue-dark); <ins> --selection-color: var(--blue-light-translucent); </ins> --success-color: var(--green-light); --success-background: var(--green-dark); --danger-color: var(--red-light);

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:
**Instructions:** Write a function `mergeSort` which takes an array of integers as input and returns an array of these integers in sorted order from least to greatest. A good way to implement this is to write one function, for instance `merge`, which is responsible for merging two sorted arrays, and another function, for instance `mergeSort`, which is responsible for the recursion that produces single-item arrays to feed into merge. Good luck! <del> **Note:** We are calling this function from behind the scenes; the test array we are using is commented out in the editor. Try logging `array` to see your sorting algorithm in action! </del> # --hints-- `mergeSort` should be a function.

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 challenge description says there is not a commented out array there I wouldn't know how to change that other than deleting the paragraph entirely Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: Google Chrome Browser Version: 87.0.4280.77 Operating System: iOS 14.1 If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):
Description is wrong. Code placeholder was changed in - test array moved from comment to function call argument, but description stayed same without any edits.
This seems quite reasonable. I will open this up to first timers
An issue with the help wanted or first timers only label is open for contribution. The first comprehensive PR created will be reviewed and merged. If you would like to contribute, and have not read the contributors docs, please do so here: If you have any issues with contributing, be sure to join us on the , or on the
I would Like to contribute to your cause i am a first timer and have not yet worked in big project but i want to start by freecodecamp
thank you for wanting to contribute, this issue has already a linked PR. Take a look at other issues with label "first timers only" and "help wanted" that do not have already a linked PR
Change:
.price { text-align: right; <del> width: 25% </del> <ins> width: 25%; </ins> } /* FOOTER */

Discussion:
.price { text-align: right; width: 25% } Line 156 of the file within the freeCodeCamp curriculum. Impact: While functionally harmless in modern browsers, adding the semicolon improves code consistency and maintainability. Discussion: The comments on the issue highlight: It's likely a typo since other CSS rules in the challenge have semicolons. While not strictly an error, adding the semicolon for consistency is recommended. to '' on 'Code Edit' down to 'Line 156' ; at last character The default code has not ; sign In css part It does look like a typo: ! ! Device: Laptop OS: Windows 11 Browser: edge Current Status: The issue is labeled as help wanted and type: bug. Multiple pull requests have been linked that might fix this issue: Update by sharabshrestha99 fix(curriculum): typo error on css file by brunades Additional Notes: A comment offers a reference for further details on semicolons in CSS:
it's not necessarily an error, the last rule doesn't need semicolon, but it could be for consistency yeah The list of files in which this is present is (44 files): I will open this for contribution.
Some additional context on last-rule semi-colons for those who might be interested: ! But I agree that omitting it is inconsistent with other steps and so it was likely a typo and we should fix it for consistency.
is this still open? I would like to contribute
hey We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue. Issues labeled with or are open for contributions. Please make sure you read . We prioritize contributors following the instructions in our guide. Join us in or if you need help contributing - our community will be happy to assist you.
Change:
# --seed-- <ins> ## --after-user-code-- ```js const _test_bob = new Person('Bob Ross'); ``` </ins> ## --seed-contents-- ```js const Person = function(firstAndLast) { <del> // Only change code below this line // Complete the method below and implement the others similarly </del> this.getFullName = function() { return ""; };

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:
] ], "type": "hike", <del> "challengeType": 6 </del> <ins> "challengeType": 6, "nameEs": "Herramientas de desarrollo de Chrome: Línea de tiempo", "descriptionEs": [ "El panel Línea de tiempo te permite grabar y analizar toda la actividad en tu aplicación mientras se ejecuta. Es el mejor lugar para comenzar a investigar los problemas de rendimiento que percibas en tu aplicación. ", "Al igual que usas el panel de red para averiguar cuánto tiempo toman las partes de la página para comunicarse con el Internet y utilizas fuentes para averiguar dónde algo se daña, puedes utilizar la línea de tiempo para averiguar con más detalle lo que está pasando y el tiempo que tarda.", "En el panel línea de tiempo grabas acontecimientos con el pequeño botón circular. Puedes grabar (o capturar) pilas, perfiles js, la memoria y la presentación (CSS). ", "Al grabarlos (capturalos), quedan en pequeños y detallados registros que puedes organizar de muchas maneras para que se adapten a tus necesidades.", "Todo esto te ayudará a entender mejor lo que está pasando en el navegador, en particular cuáles eventos requieren más memoria del navegador.", "Cuando uses la línea de tiempo, recuerda que debes utilizar el botón 'limpiar' (clear) para iniciar la grabación y antes de cargar la página, dependiendo de lo que desees capturar.", "Te darás cuenta de que cuando grabas eventos JS, al pulsar en el título o en una 'llamada a una función' al lado izquierdo, te llevará al documento y a la línea de esa función en el panel de recursos. ", "Revisa documentación más detallada aquí: developers.google.com/web/tools/profile-performance/evaluate-performance/timeline-tool" ] </ins> }, { "id": "56b15f15632298c12f315180",

Discussion:
[x] Chrome Dev Tools: Elements [x] Chrome Dev Tools: Network [x] Chrome Dev Tools: Sources [x] Chrome Dev Tools: Timeline [x] Chrome Dev Tools: Profiles [x] Chrome Dev Tools: Resources [x] Chrome Dev Tools: Audits [x] Chrome Dev Tools: Console [x] Chrome Dev Tools: Summary
Working on it