Skip to content
This repository has been archived by the owner on Jun 3, 2020. It is now read-only.

Implement the AppCache for Beginners tutorial. #34

Merged
merged 1 commit into from
Sep 29, 2012
Merged

Conversation

jason9t
Copy link
Contributor

@jason9t jason9t commented Sep 28, 2012

No description provided.

// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.

// This is a port of "A Beginner's Guide to using the Application Cache" to
Copy link
Contributor

Choose a reason for hiding this comment

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

Ok, totally pedantic, but can you capitalize "using"?

Copy link
Contributor

Choose a reason for hiding this comment

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

Done.

@jjinux
Copy link
Contributor

jjinux commented Sep 28, 2012

Other than those very few, tiny comments, LGTM.

@jjinux
Copy link
Contributor

jjinux commented Sep 29, 2012

I want to get this in today, so I'm going to merge it and then submit the fixes myself :)

jjinux added a commit that referenced this pull request Sep 29, 2012
Implement the AppCache for Beginners tutorial.
@jjinux jjinux merged commit 2b5b682 into master Sep 29, 2012
@jjinux
Copy link
Contributor

jjinux commented Sep 29, 2012

#35

@jason9t
Copy link
Contributor Author

jason9t commented Oct 1, 2012

Thanks JJ!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants