freeCodeCamp / classroom

BSD 3-Clause "New" or "Revised" License
144 stars 120 forks source link

Updated data processing and added initial details dashboard. #476

Closed ngillux closed 1 year ago

ngillux commented 1 year ago

Checklist:

utsab commented 1 year ago

@ngillux After merging PR #473, the current PR is showing merge conflicts in two files. Can you take a look at the merge conflicts and resolve them?

ngillux commented 1 year ago

@utsab I resolved the merge conflict.

In the api_processor.js file in the formattedStudentData() function I had changed a property name (changed 'name' to 'blockName') and added a property to the getSuperBlockJsons(superblockURLS) function which I thought would be needed for the details page in this PR. You should be able to see/test the updated changes in a Codespace now.

Let me know if there's any issues, or if you have any optimization suggestions.