Steven Parker185,909 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.