rbeer / nodebb-plugin-smoothshorts

Seamless short URLs for topic and post links in NodeBB.
MIT License
5 stars 2 forks source link

Feature Request: Add ability to remove the /ss/ portion of the URL #7

Closed GitPuffy closed 8 years ago

GitPuffy commented 8 years ago

I would find it advantageous if I could have the /ss/ portion of the URL removed from the provided link. Only from the portion of the URL that the user gets in his copy buffer. Not necessarily from how your software works on the backside

Let me attempt to explain what I do

Thank you.

rbeer commented 8 years ago

Alright, this should do the trick. :smile:

You can pull the 0.3.0-pre branch to test the feature. Feedback is appreciated. Especially on whether the automatic settings update is safe. Your former 'Forced Domain' setting should show up in the 'URL Format' field (given that the checkbox was checked, otherwise the value is ignored).

GitPuffy commented 8 years ago

Your new changes work like a champ! Thank you. I'll keep testing new versions as you put them out.

And yes, my previous Forced Domain setting was retained. It actually contained my short domain the "/ss" path and then ":hash".

I've since changed it to just my shortdomain/:hash and it works just as I was hoping.

rbeer commented 8 years ago

It actually contained my short domain the "/ss" path and then ":hash".

Yup, that was my plan. Keeping the domain/ss/:hash thing as default and let the admin change it as deemed fit. Thanks for testing! :smiley: :

rbeer commented 8 years ago

published in 0.3.0 / 1386a2a