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

Fixed #21847 -- Clarify where ContactForm was defined #2206

Closed
wants to merge 2 commits into from
Closed

Fixed #21847 -- Clarify where ContactForm was defined #2206

wants to merge 2 commits into from

Conversation

keybits
Copy link

@keybits keybits commented Jan 23, 2014

No description provided.

@@ -96,6 +96,7 @@ The standard pattern for processing a form in a view looks like this:

def contact(request):
if request.method == 'POST': # If the form has been submitted...
# ContactForm below was defined the previous section of the docs
Copy link
Member

Choose a reason for hiding this comment

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

I'd suggest "ContactForm was defined in the the previous section"

@timgraham
Copy link
Member

Thanks, in the future feel free to amend the original commit and force push. I've squashed the commits and merged in 3b869b0.

@timgraham timgraham closed this Jan 23, 2014
@keybits
Copy link
Author

keybits commented Jan 23, 2014

@timgraham Thanks for the help. Just learning how to contribute, so your suggestions gratefully received.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants