[fix][s]: add check for resources without an organization

datopian / ckanext-opendatani

GNU Affero General Public License v3.0

4 stars 5 forks source link

[fix][s]: add check for resources without an organization #13

Closed mpolidori closed 4 years ago

mpolidori commented 4 years ago

Okay. Hopefully this'll be the last review...

I added the check for organization. If there's no org, it will output None for dataset_organization and dataset_organization_url. I did quite a bit of testing with all of the live API data (including private datasets). Should be good to go. If it all looks good, I can deploy it any time over the weekend. I'll be around.

mpolidori commented 4 years ago

@zelima Another thing I noticed is that all resources are stored internally. I don't know if they plan on using any external resources, but I was wondering if we should keep this field now. Let me know what you think. It would definitely be useful if they plan on using some external resources.