Replace alert() with console.log()
parambirs opened this issue · 2 comments
parambirs commented
Hi there,
In some of the samples, alert()
is used for printing debugging info. Would it be better to replace such statements with console.log
?
I'm happy to raise a pull request for some of the changes I've made while trying out the samples on my machine.
Thanks
Param
DanielRosenwasser commented
Hi @parambirs - I think the sample was acquired from the playground/walkthrough where code would get executed in the browser. alert
is more explicit there for users than console.log
. We'd be happy to take a pull request!
DanielRosenwasser commented
Closed by #66.