Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#16595 closed Cleanup/optimization (fixed)

SessionBase docs don't mention pop()

Reported by: wilfred@… Owned by: nobody
Component: Documentation Version: 1.3
Severity: Normal Keywords: SessionBase pop
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


The functionality has been there since 2007, and it would be nice to mention that you can do request.session.pop('foo').

Attachments (1)

mention_pop_in_docs.diff (448 bytes) - added by wilfred@… 5 years ago.
Documentation patch to mention pop()

Download all attachments as: .zip

Change History (4)

Changed 5 years ago by wilfred@…

Documentation patch to mention pop()

comment:1 Changed 5 years ago by bpeschier

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Ready for checkin

Treating this as a spelling correction, does not really need any more text.

comment:2 Changed 5 years ago by timo

  • Resolution set to fixed
  • Status changed from new to closed

In [16628]:

(The changeset message doesn't reference this ticket)

comment:3 Changed 5 years ago by timo

In [16629]:

[1.3.X] Fixed #16595 - Add pop() to session docs; thanks wilfred.

Backport of r16628 from trunk.

Note: See TracTickets for help on using tickets.
Back to Top