solution #1554
solution #1554
Conversation
There was a problem hiding this comment.
Great job on implementing the task requirements! 🎉 The code meets the functionality and constraints as specified. Just a small suggestion: since the sortList and getEmployees functions use the list variable from the outer scope, you can simplify your function calls by removing the list parameter. Keep up the great work and happy coding! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| })); | ||
| } | ||
|
|
||
| sortList(list); |
There was a problem hiding this comment.
The sortList function is called with a parameter list, but the function definition does not accept any parameters and uses the list variable directly from the outer scope. Consider removing the parameter from the function call.
| } | ||
|
|
||
| sortList(list); | ||
| getEmployees(list); |
There was a problem hiding this comment.
The getEmployees function is called with a parameter list, but the function definition does not accept any parameters and uses the list variable directly from the outer scope. Consider removing the parameter from the function call.
[DEMO LINK] (https://kami2693.github.io/js_task_parse_list_DOM/)