search

IATI / ckanext-iati

CKAN extension for the IATI Registry
http://iatiregistry.org
9 stars 6 forks source link

Redirects to update the publishers wheeler -> lbs not working #369

Closed adrianoamaral closed 3 years ago

adrianoamaral commented 3 years ago

The publisher id of a publisher, it used to be wheeler(https://iatiregistry.org/publisher/wheeler) and now it’s lbs (https://iatiregistry.org/publisher/lbs). Usually when I make this change it appears in the Redirects tab, however I am not able to see it there. I did click the Update redirects button.

Maybe Ckan needs to be restarted? Usually, I email Swaroop to restart Ckan after I make such a change.

The publisher needs to publish their data this week, but they get an error in AidStream related to invalid publisherid when they try to use the new lbs id. Since I can’t see the wheeler->lbs change in the redirects I was not sure if the issue is in the Registry or in AidStream. Hence, I wanted to check if the publisher id was indeed correctly changed.

adrianoamaral commented 3 years ago

Not working when hit "Redirects Update"

Screenshot 2021-08-18 at 14 40 56
andreaszenasidi commented 3 years ago

@gtkChop @adrianoamaral do you have any update on this issue? Many thanks.

adrianoamaral commented 3 years ago

Swaroop is working on it! Just answered me….

Adriano Amaral about.me/adriano_amaral Enviado via IPhone

Em 19 de ago. de 2021, à(s) 15:18, Andi Szenasi @.***> escreveu:

 @gtkChop @adrianoamaral do you have any update on this issue? Many thanks.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

ss-bhat commented 3 years ago

@adrianoamaral @andreaszenasidi

This is fixed.

The problem is in the activity table. The database table structure has been changed in the new ckan. Updated the code and works well.

Please let me know if have any questions. Thanks

andreaszenasidi commented 3 years ago

@gtkChop looks good! Thank you!

andylolz commented 3 years ago

The database table structure has been changed in the new ckan.

^^ This suggests CKAN has recently been updated. Is that the case? Thanks

andreaszenasidi commented 3 years ago

The database table structure has been changed in the new ckan.

^^ This suggests CKAN has recently been updated. Is that the case? Thanks

@gtkChop @jodiegardiner Could you please address this question. Thanks.

jodiegardiner commented 3 years ago

The current CKAN version of the repository is 2.9.3 (3 days ago).

We currently do point releases this like this pretty quickly after CKAN releases them and once we have integrated them into our stack. This one has been out since May: https://docs.ckan.org/en/2.9/changelog.html

The changes to the Activity Stream tables came in 2.9.0 and we were patching the new Activity Stream system since then. This commit removes the need for our patch and was simply adjacent to the 2.9.3 upgrade.

So in short, yes, CKAN has been recently updated but only a point release so mostly bugfiixes and security.