This is mostly a house cleaning patch. I'm putting it up in preparation for some other patches I'm working on to extend googlenews.

CommentFileSizeAuthor
#9 googlenews-n1265766-9.patch5.59 KBJordanmt
#3 googlenews-n1265766-3.patch6 KBdamienmckenna
googlenews-break-out-admin.patch1.93 KBrwohleb

Comments

rwohleb’s picture

rwohleb
he/him
Primary language English
commented
Status: Active » Needs review
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Reviewed & tested by the community

Yes, and will need to be backported to D6.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Title: Break out admin page » Move the pages into separate files
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new googlenews-n1265766-3.patch6 KB

Lets separate out both files while we're at it.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Needs work
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Needs review
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Fixed » Patch (to be ported)

Needs to be backported to D6.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Jordanmt’s picture

Jordanmt commented
Issue summary: View changes
Status: Patch (to be ported) » Needs review
StatusFileSize
new googlenews-n1265766-9.patch5.59 KB

I'm working to backport a number of the D7 patches to D6 and figured this was a good place to start, as many of the other patches refer to this file structure. This patch separates the module in the same manner as #3 and replicates some of the other minor changes.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

@jordanmt: Sorry for not following up on this. Are you still interested in improving the module with backports of D7 features?

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Closed (won't fix)

The D6 version is no longer supported. Sorry folks.