search

freedomofpress / securethenews

An automated scanner and web dashboard for tracking TLS deployment across news organizations
https://securethe.news
GNU Affero General Public License v3.0
102 stars 25 forks source link

Bumps django-analytical 2.2 -> 2.5 #189

Closed conorsch closed 5 years ago

conorsch commented 5 years ago

We were pinning 2.2 because that worked well enough under Django 1.x. Under 2.1, however, the analytics integration throws an error on a bool type conversion. We saw the same error with the Django upgrade on another website, and updating analytical to 2.5 resolved it, so doing the same here.

conorsch commented 5 years ago

Confirmed this patch resolves compatibility with Django 2.1 on a sandbox server. Ready for review!