Skip to content

feat(curriculum): Prototype drafted for Workshop – Smart Campus Directory#1161

Open
Rudro4038 wants to merge 2 commits into
freeCodeCamp:mainfrom
Rudro4038:Rudro4038/feat
Open

feat(curriculum): Prototype drafted for Workshop – Smart Campus Directory#1161
Rudro4038 wants to merge 2 commits into
freeCodeCamp:mainfrom
Rudro4038:Rudro4038/feat

Conversation

@Rudro4038

@Rudro4038 Rudro4038 commented May 17, 2026

Copy link
Copy Markdown

Checklist:

References #64117

@jdwilkin4 jdwilkin4 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Higher order functions haven't been taught yet.
If this is meant to go inside the objects module, then you will need to refactor it to not use filter, map, sort, etc.

If this is meant to be inside of the higher order function module, then please let us know so the issue can be updated accordingly

@@ -0,0 +1,203 @@
{

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please create a PR in the freecodecamp CDN repo, if you haven't already done so to add this data there.

If you have already created a PR and had it approved and merged in, then this file can be removed.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's confirm the draft, then I'll would PR finalized data.json

Comment on lines +1 to +5
//this data will be fetched from CDN
const fs = require("fs");
const data = fs.readFileSync("data.json", "utf8");
const rawData = JSON.parse(data);
// console.log(rawData);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please create a PR in the freecodecamp CDN repo, to add the data.json.

If you have already created a PR and had it approved and merged in, then this will need to be updated to come from the CDN instead.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After confirming the draft these lines will be removed.

@Rudro4038

Copy link
Copy Markdown
Author

Higher order functions haven't been taught yet.
If this is meant to go inside the objects module, then you will need to refactor it to not use filter, map, sort, etc.

It is going into JS objects module, so I'm removing filter, map, sort and other higher order fucntions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants