Skip to content

Solution - #1875

Open
HiljSV wants to merge 1 commit into
mate-academy:masterfrom
HiljSV:develop
Open

HiljSV wants to merge 1 commit into
mate-academy:masterfrom
HiljSV:develop

Conversation

@HiljSV

@HiljSV HiljSV commented Aug 25, 2026

Copy link
Copy Markdown

@mateacademy-ai-mentor mateacademy-ai-mentor left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review Summary

I am approving your solution because it meets the core requirements: you locate the .dashboard table, iterate over the people array, create rows with six cells (name, gender, born, died, age, century), compute age as died - born, and compute century using Math.ceil(died / 100), then append the rows to the table body. Functionality appears correct and complete for the task’s main goals.

Minor improvement (non-blocking): the task mentions using console.log to inspect the people array. Consider adding console.log(people) near the start of src/scripts/main.js (around line 354 where you initialize the table generation) to make your debugging process explicit and align with the instruction. Everything else looks solid—nice work implementing the calculations and DOM updates cleanly.


✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨

Footnotes

  1. Rate AI review example

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants