Skip to content

A chapter on authentication and authorization#53

Merged
oinopion merged 2 commits into
DjangoGirls:masterfrom
kampfschlaefer:chapter_security_authentication_and_authorization
Jul 26, 2014
Merged

A chapter on authentication and authorization#53
oinopion merged 2 commits into
DjangoGirls:masterfrom
kampfschlaefer:chapter_security_authentication_and_authorization

Conversation

@kampfschlaefer
Copy link
Copy Markdown
Contributor

This is a first rough version. There is room for improvement and proof-reading.

This implements #47

This is a first rough version. There is room for improvement and proof-reading.
@keikoro
Copy link
Copy Markdown

keikoro commented Jul 26, 2014

Heaven't read through it yet, but: aaaawwwwesome! ⭐

@oinopion
Copy link
Copy Markdown
Contributor

If some native speaker can looks at the language, that would be swell :-)

Comment thread authentication_authorization/README.md Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

should might, should see or might see? not sure.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed in my local next version :-)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

 - Show the username when logged in
 - Show a "log out" link when logged in
 - Redirect to the index when logging out

and fix some typo on the way.
oinopion added a commit that referenced this pull request Jul 26, 2014
…ation_and_authorization

A chapter on authentication and authorization
@oinopion oinopion merged commit ae24026 into DjangoGirls:master Jul 26, 2014
@oinopion
Copy link
Copy Markdown
Contributor

Thanks. I've merged it. Other language issues will be handled separately.

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.

4 participants