fix variable naming for matching the document

char0n / django-brevisurl

Django app for shortening urls

Other

9 stars 10 forks source link

fix variable naming for matching the document #21

Closed tzangms closed 8 years ago

tzangms commented 10 years ago

If I guess right, this setting should be change to BREVISURL_ prefix, like others.

from

SHORTURL_LOCAL_BACKEND_ORIGINAL_URL_MAX_LENGTH

BREVISURL_LOCAL_BACKEND_ORIGINAL_URL_MAX_LENGTH

char0n commented 10 years ago

This totally breaks backward compatibility. Can you provide solution to support new naming but also support old one with deprecation notification notifying user that he is supposed to switch to new naming ?

tzangms commented 10 years ago

Sure, let me try it.

char0n commented 9 years ago

Do we have any progress here ? Will you provide pull request with backward compatible solution ?

teddywing commented 8 years ago

Added an update for backwards compatibility: #24