-
Notifications
You must be signed in to change notification settings - Fork 40.7k
Open
Labels
Status: Needs ReviewThis issue/PR needs an initial or additional reviewThis issue/PR needs an initial or additional review
Description
Prerequisites
- I have thoroughly read and understand The Odin Project Contributing Guide
- I have searched the existing issues to ensure this improvement hasn't been suggested before (please have a look through our open issues list to make sure)
- The title of this issue follows the
<Location of the improvement>: <Brief description of the improvement>
format, e.g.Exercises: Add exercise on XYZ
- (Optional) I am interested in working on this issue and would like to be assigned to it
Improvement Description
As a TOP student I successfully did this exercise by using two for loops – though it was not pretty.
After checking the solution, I see that both array.forEach
and array.Includes()
would make this a whole lot easier, and I'm thankful I learned that.
But I can't help but think that it would be a better learning experience if I would get a hint about those methods beforehand in the hints section.
I'm pretty sure I wasn't exposed to the either forEach
or Includes
earlier in the course, but I might be wrong.
Acceptance Criteria
- Include a subtle hint about the
forEach
andIncludes
methods in the hints section.
(Optional) Additional Comments
No response
Metadata
Metadata
Assignees
Labels
Status: Needs ReviewThis issue/PR needs an initial or additional reviewThis issue/PR needs an initial or additional review