search

EnviDat / ckanext-restricted

CKAN extension for restricting access to resources
GNU Affero General Public License v3.0
7 stars 26 forks source link

CKAN 2.7 #3

Closed m431m closed 6 years ago

m431m commented 6 years ago

Hi,

Is ckan-restricted compatible with CKAN 2.7 release?

m431m

espona commented 6 years ago

Sorry, it is not compatible at the moment.

m431m commented 6 years ago

Does it requires a lot of work?

espona commented 6 years ago

Since I have no experience with CKAN 2.7 it is hard to say. I will eventually make it compatible but I can't tell when I will have time.

m431m commented 6 years ago

I take a look.

m431m commented 6 years ago

Ok, I did, there's not much to be changed. I prepare you the pull-request

espona commented 6 years ago

Oh, really? That's great. Will it still work with previous versions? Should I do a branch for CKAN 2.7? Sorry, I am not that experienced with GitHub, I do not know how to handle this. Many thanks!!

m431m commented 6 years ago

Yes I guess it still work with previous versions. It is small changes, some bug corrections and improvement (unicode errors and import errors) on the master version. I check that all is in good on our instance and I open the pull-request with the proposed changes.