> I'm happy enough with this patch. It's not an ideal solution for all
> problems, but it's a reasonable path of least resistance for the
> current issues.
Agreed, it's a compromise for the time being.
> Is #7977 something you want to complete before the merge or let it
> wait.
It's obviously more of an annoyance than a critical bug, just
something one of our users encountered and reported; as such it can
wait until we come up with a better solution.
GeoRSS support (#6547) is a more important feature, and in a similar
predicament. Essentially, there needs to be hooks to customize the
low-level XML elements in RSS and Atom feeds so that location elements
may be inserted. At the moment, I cannot see how this could be done
without duplicating and modifying large portions of
`django.contrib.syndication`. [1] I think there's enough work on the
core devs plate for 1.0 besides rewriting syndication, so this one can
wait as well -- we've got a deadline to meet.
> Anyway, I'm obviously +1 on merging this. The code is excellent,
> you've worked well with the existing maintainers over the lifetime of
> the branch and have actively improved the core code in the process. I
> think you've achieved your goals here.
Thanks for the compliment, your support and feedback, it's been fun :)
-Justin
[1] See Robert Coup's analysis:
http://code.djangoproject.com/ticket/6547#comment:5--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Django developers" group.
To post to this group, send email to
django-developers@...
To unsubscribe from this group, send email to
django-developers+unsubscribe@...
For more options, visit this group at
http://groups.google.com/group/django-developers?hl=en-~----------~----~----~----~------~----~------~--~---
