Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

docs(dart): update BASICS intro #1410

Merged
merged 2 commits into from
May 18, 2016
Merged

Conversation

kwalrath
Copy link
Contributor

Also copy edited the TS version a bit.

NOTE: The TS version (and thus the Dart version) talks about "Reference" and "Resources" sections that don't exist. (API Reference does, but not Reference, as such.)

@kwalrath
Copy link
Contributor Author

@wardbell do you want me to make the section descriptions match what's really there?

@wardbell
Copy link
Contributor

@kwalrath Please do :-)

A running version of that sample is at
https://angular-examples.github.io/architecture/.

A few early chapters are written as tutorials and are clearly marked as such.
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not include this paragraph (L15-18) in the TS version?

Copy link
Contributor Author

@kwalrath kwalrath May 18, 2016

Choose a reason for hiding this comment

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

It's the "live link" vs. "sample repo" difference. I could've made it a variable and probably should.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, right. I just feel that we should minimize differences between the language versions. The added remark will be useful to TS and JS users too :). If you don't want to define a variable, you could just drop the last sentence (L18) and put L15-17 in the TS version. It's up to you.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It actually is in the TS & JS pages, but they have an all-plunkr solution where they can change the example. So the text needs to be different.

@chalin
Copy link
Contributor

chalin commented May 17, 2016

Other than the one inlined comment made about moving a paragraph to the TS version, it LGTM.

kwalrath added 2 commits May 18, 2016 10:52
Also copy edited the TS version a bit.

NOTE: The TS version (and thus the Dart version) talks about "Reference" and "Resources" sections that don't exist. (API Reference does, but not Reference, as such.)
@kwalrath
Copy link
Contributor Author

About to merge...

@kwalrath kwalrath merged commit a595eb8 into angular:master May 18, 2016
@kwalrath kwalrath deleted the guide_index branch May 18, 2016 20:43
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.

4 participants