Fix typo, so it actually does invoke the setter #981

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.

Sign up for GitHub

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

Jump to bottom

Merged

iliakan merged 1 commit into javascript-tutorial:master from CrazyTim:CrazyTim-patch-1

May 15, 2019

Contributor

CrazyTim commented May 15, 2019 •

edited

Loading

user = new User(""); // Name too short.

Doesn't show an alert as intended.


          Fix typo, so it actually does invoke the setter

c5fab43

CLAassistant commented May 15, 2019 •

edited

Loading

All committers have signed the CLA.

iliakan merged commit d11039e into javascript-tutorial:master

CrazyTim deleted the CrazyTim-patch-1 branch

May 15, 2019 12:13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet