Feat: add prototype for workshop-sensor-burst-analyzer#1071
Feat: add prototype for workshop-sensor-burst-analyzer#1071sebacodes wants to merge 9 commits intofreeCodeCamp:mainfrom
Conversation
|
Is this ready for review? If it is ready, please bring it out of draft status |
- Change summarizedData from array `[[],[],[]]` to object `{"min":[],"max":[],"avg":[]}`
to match prototype specifications
- Round average values in summarizeSensorData for cleaner output
- Use object property notation (summarizedData.min.push, and so on.) instead of array indices
- Fix minor typos and improve comments
|
Status changed, ready for review |
jdwilkin4
left a comment
There was a problem hiding this comment.
I am not sure I understand the learning objectives in this workshop.
Is it to practice arrays?
working with loops?
something else?
where is this supposed to go in the curriculum?
I am going to tag Naomi to review this 👍🏾
- file do not belongs to the root directory - file goes on fullstack-cert/js-projects/sensor-burst-analyzer/ directory but is not being use
|
Are there answers for my questions here? |
|
Hey @sebacodes! Thanks for your patience on this one — I've been reviewing everything and I owe you some clarity. 🙂 After looking at where this workshop sits in the curriculum, I realised the original user stories were asking for The new user stories are:
Your confusion about the approach makes complete sense given the original spec wasn't aligned with the curriculum — that was on me, not you. The good news is the rework is fairly focused: swap out the existing functions for these five, each built around one of the lecture methods. Happy to answer any questions in Discord! |
Checklist:
Update index.md)relates to freeCodeCamp/freeCodeCamp#64114