[tinder] feed syndication to use the new class based views instead

RESOLVED INVALID

Status

Webtools
Elmo
P3
normal
RESOLVED INVALID
7 years ago
6 years ago

People

(Reporter: peterbe, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

7 years ago
The non-class based use of Feed syndication is deprecated (only a warning) in Django 1.3. Consider replacing all feeds in tinder with class based ones. 

Also, there are no unit tests for any feeds. Especially not for URLs like 'tinder.views.feed' which is based on an import from the django.contrib.
Priority: -- → P3

Comment 1

6 years ago
Peter, I don't understand this bug, can you rephrase what exactly is the problem?
(Reporter)

Comment 2

6 years ago
Hmm... 

To begin with, we don't use Django 1.3 any more. 
Secondly, we are apparently using the correct import, i.e.
`from django.contrib.syndication.feeds import Feed`
Perhaps we didn't do that before???

I get no warnings any more on this.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.