char0n / django-brevisurl

Django app for shortening urls
Other
9 stars 10 forks source link

Make updated max length settings variable backwards compatible #24

Closed teddywing closed 8 years ago

teddywing commented 8 years ago

An extension of #21 to change the SHORTURL_LOCAL_BACKEND_ORIGINAL_URL_MAX_LENGTH setting to BREVISURL_LOCAL_BACKEND_ORIGINAL_URL_MAX_LENGTH to match the documentation.

This change allows both settings to work for backwards compatibility as proposed here: https://github.com/CodeScaleInc/django-brevisurl/pull/21#issuecomment-50590660.

Had some trouble trying to write tests for this and could use some help there if we want to make sure there's coverage for this.

char0n commented 8 years ago

Cool thanks.