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

docs(guide/concepts) updating concepts to reflect the actual example #7243

Closed
wants to merge 1 commit into from
Closed

docs(guide/concepts) updating concepts to reflect the actual example #7243

wants to merge 1 commit into from

Conversation

plentz
Copy link
Contributor

@plentz plentz commented Apr 25, 2014

The actual "A first example: Data binding" section in concepts actually doesn't do anything with the required directive, so I've removed the part of the section that explains it to avoid confusion.

The "A first example: Data binding" section it implies that the required directive is actually
doing something, but it isn't.

I just removed the parts the refer to the required directive to avoid confusion.
@mary-poppins
Copy link

Thanks for the PR! Please check the items below to help us merge this faster. See the contributing docs for more information.

  • Uses the issue template (#7243)

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

@plentz
Copy link
Contributor Author

plentz commented Apr 25, 2014

@btford
Copy link
Contributor

btford commented Jul 23, 2014

I like this change.

@btford btford self-assigned this Jul 23, 2014
@btford
Copy link
Contributor

btford commented Jul 23, 2014

Landed as 1a2ed70. Thanks!

@btford btford closed this Jul 23, 2014
@plentz
Copy link
Contributor Author

plentz commented Jul 23, 2014

@btford 👍

@plentz plentz deleted the docs_guide_concepts branch July 23, 2014 18:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants