Skip to content

docs(WebWorkers): Add overview of WebWorker design#4103

Closed
jteplitz wants to merge 1 commit into
angular:masterfrom
jteplitz:documentation
Closed

docs(WebWorkers): Add overview of WebWorker design#4103
jteplitz wants to merge 1 commit into
angular:masterfrom
jteplitz:documentation

Conversation

@jteplitz
Copy link
Copy Markdown
Contributor

@jteplitz jteplitz commented Sep 9, 2015

No description provided.

@jteplitz jteplitz added comp: docs action: review The PR is still awaiting reviews from at least one requested reviewer labels Sep 9, 2015
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are a master of suspense ;)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Epic comment (lol)

@yjbanov
Copy link
Copy Markdown
Contributor

yjbanov commented Sep 9, 2015

/cc @naomiblack @kwalrath

This is a very user-friendly design doc. We should turn it into a page on angular.io.

@yjbanov yjbanov added pr_state: LGTM action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Sep 9, 2015
@jteplitz jteplitz removed the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label Sep 9, 2015
@webteckie
Copy link
Copy Markdown

fyi...there's an incomplete sentence in the introduction: "By offloading as much code as possible to the"

@jteplitz
Copy link
Copy Markdown
Contributor Author

jteplitz commented Sep 9, 2015

Fixed. Thanks Carlos!

@jteplitz jteplitz closed this in 2daf2ee Sep 9, 2015
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be 'loader.js' ?

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.

5 participants