Allowing forms to submit responses

Review Request #8784 — Created Feb. 26, 2017 and discarded

Information

Djblets
master

Reviewers

In a case that we want a form to submit its own response (with different headers or a different return path).

This is necessary for python-social-auth disconnects, as those require a POST request to a specific URL.

None, except that it seems to work.

Description From Last Updated

"Submit" should be "return"

brenniebrennie

Please add documentation to save() to indicate that it can return either HttpResponse or None.

brenniebrennie

This should be response or form_response. Python variable names (except constants) are always snake_case.

brenniebrennie

Blank line between these.

brenniebrennie

No parenthesis needed.

brenniebrennie
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        djblets/configforms/views.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        djblets/configforms/views.py
    
    
  2. 
      
brennie
  1. 
      
  2. Show all issues

    "Submit" should be "return"

  3. Show all issues

    Please add documentation to save() to indicate that it can return either HttpResponse or None.

  4. djblets/configforms/views.py (Diff revision 1)
     
     
    Show all issues

    This should be response or form_response. Python variable names (except constants) are always snake_case.

  5. djblets/configforms/views.py (Diff revision 1)
     
     
     
    Show all issues

    Blank line between these.

  6. djblets/configforms/views.py (Diff revision 1)
     
     
    Show all issues

    No parenthesis needed.

  7. 
      
brennie
david
Review request changed
Status:
Discarded