search

ckan / ckanext-issues

CKAN Issues Extension
https://github.com/ckan/ideas-and-roadmap/issues/4
MIT License
14 stars 31 forks source link

Change 'redirect' calls to 'h.redirect_to' #65

Closed argv0 closed 7 years ago

argv0 commented 7 years ago

ckan.lib.base.redirect was removed in [1], guidance is to always use h.redirect_to instead. Manifested as 503 errors in the issues UI against an install of CKAN master branch.

[1] ckan/ckan@34f3f18

davidread commented 7 years ago

Thanks for flagging this up. I fixed one you missed and changed it to use the p.toolkit.redirect instead of h.toolkit, as is best practise. This is in a new PR: https://github.com/ckan/ckanext-issues/pull/67

davidread commented 7 years ago

Ah sorry, there is only p.toolkit.redirect_to, not redirect itself.