kipusoep / UrlTracker

The Url Tracker for Umbraco

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Not allowed root "/" in old url

yonatancohen opened this issue · comments

Hi, I want to define the next rule :
capture

However i get an error that the rule is invalid... Can't i use the "/" in old url?
when I look in the DB it's save as "NULL", but if i change the value to "/" - all works fine..

Anyway for quick fixing?

The reason is because the smallest URL is being stored; this means without leading and trailing slash. That's why NULL is stored in the database.

Anyway to override that option?

I have the following urls :

Can i do that using the "Url Tracker"?

Thanks !!!!!

‫בתאריך יום ה׳, 21 במאי 2015 ב-17:34 מאת ‪Stefan Kip‬‏ <‪
notifications@github.com‬‏>:‬

The reason is because the smallest URL is being stored; this means without
leading and trailing slash. That's why NULL is stored in the database.


Reply to this email directly or view it on GitHub
#79 (comment).

Not via the UI, if it works via SQL you're lucky :-)

Ok, so i'll download the code and try to add it myself.

Thanks,

I'll try to fix this soonish :-)

I will really appreciate it!!
On Fri, May 22, 2015 at 17:27 Stefan Kip notifications@github.com wrote:

I'll try to fix this soonish :-)


Reply to this email directly or view it on GitHub
#79 (comment).

Hi, I wonder if you go into it?

Thanks,

‫בתאריך יום ו׳, 22 במאי 2015 ב-17:29 מאת ‪Yoni Cohen‬‏ <‪
yoni.n.cohen@gmail.com‬‏>:‬

I will really appreciate it!!
On Fri, May 22, 2015 at 17:27 Stefan Kip notifications@github.com wrote:

I'll try to fix this soonish :-)


Reply to this email directly or view it on GitHub
#79 (comment)
.

Fixed in 2fad9b7, not released yet though.

Ok.. So first thanks a lot !
Second, when do you think you will release it?
On Tue, May 26, 2015 at 23:32 Stefan Kip notifications@github.com wrote:

Fixed in 2fad9b7
2fad9b7,
not released yet though.


Reply to this email directly or view it on GitHub
#79 (comment).

When there's a bit more to release. You can download the solution and create a build for yourself though, no sweat.

Hi, now it's allow to add "/" at root - but doesn't do the redirect itself.

‫בתאריך יום ג׳, 26 במאי 2015 ב-23:34 מאת ‪Stefan Kip‬‏ <‪
notifications@github.com‬‏>:‬

When there's a bit more to release. You can download the solution and
create a build for yourself though, no sweat.


Reply to this email directly or view it on GitHub
#79 (comment).

It works for me though, altough I tried by setting no root node (root node set to the first option '/').
Does it work if you select no root node (first option)?

Both doesn't seem to work,

I set the root (in combo) to "/",
Enter some query "Id=123",
and the redirect path to "http://www.google.com"

‫בתאריך יום ה׳, 28 במאי 2015 ב-15:03 מאת ‪Stefan Kip‬‏ <‪
notifications@github.com‬‏>:‬

It works for me though, altough I tried by setting no root node (root node
set to the first option '/').
Does it work if you select no root node (first option)?


Reply to this email directly or view it on GitHub
#79 (comment).

Then I'm not sure what's going on to be honest...