Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Proofreading changes #29

Merged
merged 4 commits into from
Jul 16, 2014
Merged

Proofreading changes #29

merged 4 commits into from
Jul 16, 2014

Conversation

rebkwok
Copy link
Contributor

@rebkwok rebkwok commented Jul 15, 2014

Proofread chapters:

  • Django models
  • Django admin
  • Django urls
  • Django views

Made very minor modifications to wording for correct English usage.  One typo in the reference to "...settings.py" corrected.
Very minor changes to wording after proofreading.  "Go to Post..." changed to "Go to Posts..." for consistency with the image that it refers to (although of course the model itself is named "Post", but it appears in the admin page as "Posts")
@rebkwok rebkwok mentioned this pull request Jul 15, 2014
22 tasks
@olasitarska
Copy link
Member

Thanks a lot, great work! 👍 ❤️

olasitarska added a commit that referenced this pull request Jul 16, 2014
@olasitarska olasitarska merged commit 697265d into DjangoGirls:master Jul 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants