Ticket #4192 task closed fixed

Opened 3 years ago

Last modified 3 years ago

Move the content from exarkun's Twisted Web in 60 Seconds series into the Twisted Web howto

Reported by: jesstess Owned by:
Priority: normal Milestone:
Component: web Keywords: documentation
Cc: jesstess, thijs Branch: branches/web-in60-4192
Author: jesstess Launchpad Bug:

Description

Index for the content is here:  http://jcalderone.livejournal.com/50562.html

Change History

1

Changed 3 years ago by jesstess

  • branch set to branches/web-in60-4192
  • branch_author set to jesstess

(In [27894]) Branching to 'web-in60-4192'

2

Changed 3 years ago by thijs

  • keywords documentation added
  • cc thijs added

Are you going to make an additional document, or cramp it all in the existing howto (which is probably already long enough)?

4

Changed 3 years ago by jesstess

  • keywords documentation, review added; documentation removed
  • owner jesstess deleted
  • branch_author changed from Drivers Download to jesstess

Example organization of the twisted web in 60 seconds series with the first example: serving static content. The web howto has a link to the 60 seconds index which has links to all the examples, so there's minimal additional clutter on the howto index.

There are up to 17 existing examples that could go into this series. One thing that I know is missing is an attribution to exarkun.

Comments on the organization before I put in work on the rest of the examples?

5

Changed 3 years ago by TimAllen

  • owner set to TimAllen

6

Changed 3 years ago by TimAllen

  • keywords review removed
  • owner changed from TimAllen to jesstess
  1. It seems to me that if exarkun's series goes on long enough, it could replace the other Web documentation, rather than accompanying it. In particular, it seems a shame to say "This set of examples is intended to be a supplement to the Using Twisted Web tutorial", when it's so much more readable than that tutorial. Maybe you could say something like "for subjects not covered here, see the Twisted Web tutorial and the API documentation".
  2. You might want to add class="python" to all the <pre/> tags that contain Python source, not just the block of code at the end.

Apart from that, looks good to me.

7

Changed 3 years ago by jesstess

  • owner jesstess deleted
  • keywords review added

TimAllen's review comments are addressed, and parts 1-6 are now in the branch. These parts are:

  • Serving static content from a directory
  • Generating a page dynamically
  • Static URL dispatch
  • Dynamic URL dispatch
  • Error handling
  • Custom response codes

A double check on the API links would be most appreciated as part of a review. After that I'll add the middle third of the examples.

8

Changed 3 years ago by gas

link seemed fine to me when tested via firefox. Only issue i had was firefox didnt seem to take nay notice of the <pre class="python" tags and interpreted the html tags in the render_GET messages ie output on browser

def render_GET(self, request):

return "

%s

" % (calendar(self.year),)

I presume this is a browser xhmtl parsing failure.

Didnt see any attribution to exarkun - maybe add link to original and attribution in index?

9

Changed 3 years ago by jesstess

  • owner set to jesstess
  • keywords documentation added; documentation, review removed

10

Changed 3 years ago by jesstess

  • owner jesstess deleted
  • keywords documentation, review added; documentation removed

Thanks for the review, gas!

  • gas is working out lore issues, hence the above review comment about the xhtml looking fun.
  • exarkun said he'd prefer no attribution.

Back up for review!

11

Changed 3 years ago by gas

  • keywords documentation added; documentation, review removed

ok reviewed with lore all looks fine to me. maybe a suggestion for all doc should we put a last updated on them - so if newbies reading it can see if it is out of date?

12

Changed 3 years ago by gas

  • owner set to jesstess

13

Changed 3 years ago by jesstess

Parts 7-12 have now been added to the branch. These parts are:

  • handling POSTs
  • rpy scripts (or, how to save yourself some typing)
  • asynchronous responses
  • asynchronous responses (via Deferred)
  • interrupted responses
  • logging errors

As before, a double check on the API links would be most appreciated as part of a review. After that I'll add the final third of the examples.

14

Changed 3 years ago by jesstess

  • keywords documentation, review added; documentation removed
  • owner jesstess deleted

15

Changed 3 years ago by therve

  • owner set to jesstess
  • keywords documentation added; documentation, review removed

The reference to notifyFinish is wrong in interrupted.xhtml. Everything else looks great.

15

Changed 3 years ago by jesstess

(In [28251]) Correct notifyFinish link in interrupted.xhtml

refs #4192

16

Changed 3 years ago by jesstess

  • owner jesstess deleted
  • keywords documentation, review added; documentation removed

Parts 13-17 have now been added to the branch. These parts are:

  • WSGIs
  • HTTP authentication
  • Session basics
  • Storing objects in the session
  • Session endings

As before, a double check on the API links would be most appreciated as part of a review. This is the final set to review before inclusion!

17

Changed 3 years ago by therve

  • owner set to jesstess
  • keywords documentation added; documentation, review removed

Great work! Just one comment: the first Guard link in http-auth is broken.

+1!

18

Changed 3 years ago by jesstess

(In [28284]) Fix guard link.

refs #4192

19

Changed 3 years ago by jesstess

  • status changed from new to closed
  • resolution set to fixed

(In [28287]) Merge web-in60-4192

Author: jesstess Reviewers: TimAllen, gas, therve Fixes: #4192

Twisted Web in 60 Seconds, a series of short tutorials with self-contained examples on a range of common web topics, is now a part of the Twisted Web howto documentation.

20

Changed 2 years ago by <automation>

  • owner jesstess deleted
Note: See TracTickets for help on using tickets.