Closed (fixed)
Project:
Kiva
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
3 Jun 2011 at 23:42 UTC
Updated:
27 Jun 2011 at 21:02 UTC
Jump to comment: Most recent file
Title says it all, patch coming up.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | d7versionofkiva-1178316-3.patch | 16.18 KB | coderintherye |
| #1 | d7versionofkiva-1178316.patch | 14.57 KB | coderintherye |
Comments
Comment #1
coderintherye commentedHere's a patch against D6 latest but also requires my patch from http://drupal.org/node/504942#comment-4556072, going to see if I can commit a D7 release or if I will need CrookedNumber to do it.
Note this is only an upgrade of the Kiva module. I haven't yet done an upgrade of the KivaFeeds module.
Comment #2
coderintherye commentedComment #3
coderintherye commentedUpdating patch cause two of the hunks didn't apply, so instead this patch just goes directly off the current 6.x.-1.x release (no need to apply the other patch).
Comment #4
coderintherye commentedOk I created a dev branch for D7 as per the git-instructions and you can see my commit here: http://drupal.org/commitlog/commit/8700/86a046fd06f687fd7efa6944eb214f19...
However, I don't have permissions to create a release, so give it a review when you have a chance and let me know :)
To note, I'm running the D7 version on a dev machine and tested it out with lenders and partners and such, but haven't tested all features yet, so far so good though.
Comment #5
crookednumber commentedJust gave you perm to add releases. Let 'er rip!
Comment #6
crookednumber commentedLooking good. Installs cleanly. Getting a bunch of notices about undefined vars -- but that's par for the course (PHP5!).
Comment #7
crookednumber commentedFYI: Also created #1186610: Create a Drupal 7 version for kivafeeds and assigned it to CrookedNumber.
Comment #8
coderintherye commentedCoolness :)
I just added a release for 7.x, if I remember correctly I will need to come back publish the release after it goes through the system, takes a bit of time for dev releases. I'll mark this as resolved once I come back to it.
Comment #9
coderintherye commentedComment #10
coderintherye commented