[ckan-dev] CKAN API Clients

Wed May 16 19:50:24 UTC 2012

On 5/16/12 7:23 AM, "Sean Hammond" <sean.hammond at okfn.org> wrote:

>> >>> Sean,
>> >>>
>> >>> I'll fix the issue with localhost issue that you¹ve highlight.
>> >>>
>> >>> Should the functions `upload_file` and `add_package_resource` become
>> >>> methods of the CkanClient class while the helpers remain the `util`
>> >>> module? Or do you want to not have a `util` module at all?
>> >>
>> >>I'd suggest adding to main class. I'm also happy to review the
>> >>functions once done (this could substitute for getting you an upload
>> >>example).
>> >
>> >Okay, I've moved all code from the util module into the CkanClient
>>class.
>> >ckanclient.loaders.util has been deleted.
>> >
>> >See the changeset here:
>> >
>> 
>>>https://github.com/slmnhq/ckanclient/commit/d233fcb3f73644b8b505839bedc1
>>>6d
>> >3
>> >52434e946
>
>Hey Salman, did you make any more progress on this? Do you want to make
>a pull request against okfn/ckanclient and then we can review and merge
>your work?

Sean,

I've not yet been able to add test cases. Mostly because I don't
understand how to test this functionality. What do you suggest?

Salman

>
>_______________________________________________
>ckan-dev mailing list
>ckan-dev at lists.okfn.org
>http://lists.okfn.org/mailman/listinfo/ckan-dev