Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Function Basics: Extend note on not doing the MDN exercises to the second exercise bullet point #27861

Closed
2 of 3 tasks
Asartea opened this issue Apr 26, 2024 · 4 comments · Fixed by #28038
Closed
2 of 3 tasks
Assignees

Comments

@Asartea
Copy link
Contributor

Asartea commented Apr 26, 2024

Checks

Describe your suggestion

In #24895 there was a warning added to the first exercise against doing the MDN exercises, as they involve knowledge TOP hasn't touched at that point. Someone recently pointed out the second exercise article also links to those, and while I think most learners will understand the bullet point to apply to both, I don't think there's any harm in making that explicit

Path

Foundations

Lesson Url

https://www.theodinproject.com/lessons/foundations-function-basics

(Optional) Discord Name

asarta

(Optional) Additional Comments

Probably a good first issue

@wise-king-sullyman
Copy link
Member

Agree, thanks!

@MaoShizhong MaoShizhong added the Status: Help Wanted This issue can be assigned to other contributors label Apr 28, 2024
@yuliana-r
Copy link
Contributor

Happy to help with this if there's no takers!

@MaoShizhong MaoShizhong removed the Status: Help Wanted This issue can be assigned to other contributors label Apr 30, 2024
@MaoShizhong
Copy link
Contributor

Assigned, @yuliana-r!

@yuliana-r
Copy link
Contributor

Sorry about the delay on this - I figured it might look a bit better to have a single note covering both articles but if needed, happy to redo and add the note to both individually instead

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 a pull request may close this issue.

4 participants