unt-libraries / django-nomination

A Django application for nominating URLs by project.
BSD 3-Clause "New" or "Revised" License
2 stars 1 forks source link

Store https urls #87

Closed ldko closed 7 years ago

ldko commented 7 years ago

Previously we forced all URIs to be stored with http:// scheme (not allowing https). This makes changes to allow for both (and others such as ftp://) to be stored and browsed. This should fix #26 and fix #31 (both regarding unreachable branches), close #67 (http:// scheme initialized in forms), fix #80 (ftp browsing), and fix #38 (compile regex out of loop).

Please review when you get a chance @somexpert @runderwood

somexpert commented 7 years ago

Seems that things work the way they're meant to. Looks good :thumbsup: