Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

docs(tutorial/0 - Bootstrapping): Clarifying where the callback is defined #11270

Closed
wants to merge 1 commit into from
Closed

Conversation

kjarnet
Copy link

@kjarnet kjarnet commented Mar 7, 2015

Saying that the script-tag registers a callback was for me very confusing, and I think specifying that it is the code in the script that registers the callback makes this a bit clearer.

…fined.

Saying that the script-tag register a callback was for me very confusing, and I think specifying that it is the code in the script that registers the callback makes this a bit clearer.
@googlebot
Copy link

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

@kjarnet
Copy link
Author

kjarnet commented Mar 7, 2015

Signed the CLA.

@googlebot
Copy link

CLAs look good, thanks!

@googlebot googlebot added cla: yes and removed cla: no labels Mar 7, 2015
@Narretz Narretz closed this in 248b036 Mar 8, 2015
@Narretz
Copy link
Contributor

Narretz commented Mar 8, 2015

Hi, thanks for the PR! I committed another version, which I found a bit simpler. Thanks for the contribution though!

Narretz added a commit that referenced this pull request Mar 8, 2015
hansmaad pushed a commit to hansmaad/angular.js that referenced this pull request Mar 10, 2015
netman92 pushed a commit to netman92/angular.js that referenced this pull request Aug 8, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants