Welcome to the Treehouse Community
Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community!
Looking to learn something new?
Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.Start your free trial
Steven Parker228,025 Points
You're off to a good start! That's pretty much the classic implementation of this exercise.
One (minor) suggestion: If you're only going to show a message once, you can pass it directly instead of creating a variable for it:
alert("All done. ready for the message?");
It's also a "best practice" to always include a semicolon at the end of a statement.