search

n2o / url-shortener

Shorty: The URL Shortener
MIT License
17 stars 15 forks source link

Modifying pull request "Adding auto-suggesting abbreviation as per issue #3" as per issue #3 #5

Closed matlorr closed 4 years ago

matlorr commented 4 years ago

I just added an if-statement so auto generated abbreviations with the format "www.xy.z" no longer contain "www".

n2o commented 4 years ago

Thanks, but the same comments, as seen in #3 also apply here. Please put the logic for abbreviation into a separate class and call it LinkService.

n2o commented 4 years ago

Closing this in favor of #6