properly urlencode pic cached documents

[fgregg]

I made a number of changes to make our document cache much more robust. Before deploying we need to make sure that we are properly urlencoding the urls. As we do this in a few places, I'm going to wait till @reginafcompton comes back to deploy the changes to the document cache to production.

[reginafcompton]

Use the pic-staging.datamade for the new url.

[reginafcompton]

I have an initial solution for this on the pic branch. It works with pic-staging.datamade.us.

However, it seems that we actually want to create a standardized version of full_text_doc_url in django-councilmatic core. We can use the work on the pic branch as a good starting point.

datamade/django-councilmatic#136

[reginafcompton]

This PR closes this issue.

Related: The PIC changes are working well. Currently, we have LA Metro pointing to pic-staging.datamade.us. @fgregg let me know if you'd like to deploy PIC to production, and we can point LA Metro accordingly.

[fgregg]

Sounds good.

…

On Mon, Aug 14, 2017 at 3:51 PM, Regina Compton ***@***.***> wrote: This PR closes this issue. Related: The PIC changes are working well. Currently, we have LA Metro pointing to pic-staging.datamade.us. @fgregg <https://github.com/fgregg> let me know if you'd like to deploy PIC to production, and we can point LA Metro accordingly. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#184 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAgxbRURx_HpBmRpW6Kk7upQMoE1y5xHks5sYLM2gaJpZM4OdTlW> .

-- 773.888.2718

[reginafcompton]

PIC is deployed to production, and LA Metro points to pic.datamade.us.

Closing!