Any chance to make it compatible with latest DNN 9.6.2

Scippy commented 4 years ago

Hi, I love this module, it come on with very useful providers! It is one of the fundamental modules that I use for my work! The problem is that with latest DNN 9.6.2 and with 2SXC 11.04.00 it give me some problem. https://github.com/dnnsoftware/Dnn.Platform/issues/3816 https://github.com/2sic/2sxc/issues/2144 Is there any possibility of making a new release with the necessary corrections to make this module compatible again with these latest versions?

sachatrauwaen commented 4 years ago

I published a prerealse for dnn 9.6

It will be nice if you can test it and give some feedback

Scippy commented 4 years ago

Hi Sacha, I've tested but it still has some problems, for example the two problems mentioned above are still present. If it helps you can take a look at my Fork that I am currently using without any errors. https://github.com/Scippy/OpenUrlRewriter

GerritGer commented 4 years ago

Hi Scippy, i am new to github and looking for a dnn-urlrewriter. I looked your fork and downloaded the code as a zip-file. But i can't install that as an extension in dnn. What am i doing wrong? Can you give me a link to a install-version please? Thanks for your help.

Scippy commented 4 years ago

Hi Scippy, i am new to github and looking for a dnn-urlrewriter. I looked your fork and downloaded the code as a zip-file. But i can't install that as an extension in dnn. What am i doing wrong? Can you give me a link to a install-version please? Thanks for your help.

Hi, I’ll prepare an installation packages in the next days.

GerritGer commented 4 years ago

Great news! Thanks.

Von: Scippy notifications@github.com Gesendet: Freitag, 4. September 2020 08:17 An: sachatrauwaen/OpenUrlRewriter OpenUrlRewriter@noreply.github.com Cc: GerritGer gerrit.stoever@online.de; Comment comment@noreply.github.com Betreff: Re: [sachatrauwaen/OpenUrlRewriter] Any chance to make it compatible with latest DNN 9.6.2 (#22)

Hi Scippy, i am new to github and looking for a dnn-urlrewriter. I looked your fork and downloaded the code as a zip-file. But i can't install that as an extension in dnn. What am i doing wrong? Can you give me a link to a install-version please? Thanks for your help.

Hi, I’ll prepare an installation packages in the next days.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/sachatrauwaen/OpenUrlRewriter/issues/22#issuecomment-686936324 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ3R3UBLAKQNK5PPY4DFWCTSECA5JANCNFSM4PWWXI2A . https://github.com/notifications/beacon/AQ3R3UDWDRX235TG44FN3RTSECA5JA5CNFSM4PWWXI2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFDY5CBA.gif

Scippy commented 3 years ago

@GerritGer have a look here https://github.com/Scippy/OpenUrlRewriter/releases/tag/V1.4.5

GerritGer commented 3 years ago

Thanks a lot, i got it installed without problem. Now i have to figure out how it works with the ventrian news articles und property agent.

Von: Scippy notifications@github.com Gesendet: Mittwoch, 9. September 2020 17:18 An: sachatrauwaen/OpenUrlRewriter OpenUrlRewriter@noreply.github.com Cc: GerritGer gerrit.stoever@online.de; Mention mention@noreply.github.com Betreff: Re: [sachatrauwaen/OpenUrlRewriter] Any chance to make it compatible with latest DNN 9.6.2 (#22)

@GerritGer https://github.com/GerritGer have a look here https://github.com/Scippy/OpenUrlRewriter/releases/tag/V1.4.5

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sachatrauwaen/OpenUrlRewriter/issues/22#issuecomment-689631208 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ3R3UAS6P6Q7X54A7VIX4TSE6MAVANCNFSM4PWWXI2A . https://github.com/notifications/beacon/AQ3R3UD6MCFX3WW2WWJDCVLSE6MAVA5CNFSM4PWWXI2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFENO72A.gif

Scippy commented 3 years ago

@GerritGer you can find latest Ventrian PA release with OpenUrlRewriter provider here https://github.com/Scippy/Property-Agent/releases/tag/Property-Agent

GerritGer commented 3 years ago

Hi Scippy,

thanks for the info with the link.

After installing your

OpenUrlRewriter 1.4.5

Satrabel OpenUrlRewriter module

and the latest Ventrian PA release, i can open the settings page fort he url rewriter.

But after installing the PA OpenUrlRewriter provider Version 1.3.2 i get the following error when i try to open the settings page.

Do you have an idea what went wrong?

I am working with dnn 9.6.2

Gerrit

Von: Scippy notifications@github.com Gesendet: Donnerstag, 10. September 2020 09:28 An: sachatrauwaen/OpenUrlRewriter OpenUrlRewriter@noreply.github.com Cc: GerritGer gerrit.stoever@online.de; Mention mention@noreply.github.com Betreff: Re: [sachatrauwaen/OpenUrlRewriter] Any chance to make it compatible with latest DNN 9.6.2 (#22)

@GerritGer https://github.com/GerritGer you can find latest Ventrian PA release with OpenUrlRewriter provider here https://github.com/Scippy/Property-Agent/releases/tag/Property-Agent

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sachatrauwaen/OpenUrlRewriter/issues/22#issuecomment-690046884 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ3R3UGFCZOIRXZU7CP75N3SFB5XHANCNFSM4PWWXI2A . https://github.com/notifications/beacon/AQ3R3UHA4BHLAAPUXUCXRBDSFB5XHA5CNFSM4PWWXI2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFEQUPJA.gif

Scippy commented 3 years ago

Which PA version have you installed? If you've installed my latest PA 1.10.11 release you need also to update the PA Url Rewriter Provider, in that case try to replace this dll Satrabel.OpenUrlRewriter.PropertyAgent.dll.zip in the bin folder (unzip before replace)

GerritGer commented 3 years ago

Hi Scippy,

yes i have installed your latest PA release and PA URL Rewriter Provider. After replacing the dll the error disappeared. Thanks.

Now i still get a long URl

http://dnn9.hallo-bergstrasse.de/locations-finden/ausflugsziele/odenwald/sehenswertes/burg-schloss-kloster/burg-frankenstein

/locations-finden/ = pagename of the PA- module

/ausflugsziele/odenwald/sehenswertes/burg/schloss-kloster/ = Type of the Property

/burg-frankenstein = Customfield oft he Property

I would like a short URL like

http://dnn9.hallo-bergstrasse.de/burg-frankenstein

What should i do to get a url like this?

Gerrit

Scippy notifications@github.com Gesendet: Sonntag, 27. September 2020 14:05 An: sachatrauwaen/OpenUrlRewriter OpenUrlRewriter@noreply.github.com Cc: GerritGer gerrit.stoever@online.de; Mention mention@noreply.github.com Betreff: Re: [sachatrauwaen/OpenUrlRewriter] Any chance to make it compatible with latest DNN 9.6.2 (#22)

Which PA version have you installed? If you've installed my latest PA release you need to update the PA Url Rewriter Provider, in that case try to replace this dll Satrabel.OpenUrlRewriter.PropertyAgent.dll.zip https://github.com/sachatrauwaen/OpenUrlRewriter/files/5288228/Satrabel.OpenUrlRewriter.PropertyAgent.dll.zip in the bin folder (unzip before replace)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sachatrauwaen/OpenUrlRewriter/issues/22#issuecomment-699626635 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ3R3UEC4CNYNQMTGHADV63SH4S6VANCNFSM4PWWXI2A . https://github.com/notifications/beacon/AQ3R3UBFPEWAXS76KQRC4E3SH4S6VA5CNFSM4PWWXI2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFGZXJCY.gif

Scippy commented 3 years ago

Look the url after cleared the DNN server cache.

GerritGer commented 3 years ago

HI,

after clearing alle the caches i still get the long url:

http://dnn9.hallo-bergstrasse.de/locations-finden/museum/senckenbergmuseum-frankfurt

This URL also works (i get it from the protper search result list)

http://dnn9.hallo-bergstrasse.de/bergstrasse/senckenbergmuseum-frankfurt/PropertyID/49

Are there some settings which i should change?

Gerrit

Von: Scippy notifications@github.com Gesendet: Mittwoch, 30. September 2020 14:57 An: sachatrauwaen/OpenUrlRewriter OpenUrlRewriter@noreply.github.com Cc: GerritGer gerrit.stoever@online.de; Mention mention@noreply.github.com Betreff: Re: [sachatrauwaen/OpenUrlRewriter] Any chance to make it compatible with latest DNN 9.6.2 (#22)

Look the url after cleared the DNN server cache.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sachatrauwaen/OpenUrlRewriter/issues/22#issuecomment-701372890 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ3R3UHT6BIBAYTHVRBZ6DLSIMTJ5ANCNFSM4PWWXI2A . https://github.com/notifications/beacon/AQ3R3UHMBYYDRCCIPKKMGXLSIMTJ5A5CNFSM4PWWXI2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFHHBTWQ.gif

Scippy commented 3 years ago

To enable OpenUrlRewrite on PA you must disable "Do Redirect" in PA SEO settings, then clear cache. Screenshot_4

GerritGer commented 3 years ago

Hi,

„Do Redirect“ is unchecked.

If I leave „View property Title“ empty I get an url like this:

http://dnn9.hallo-bergstrasse.de/locations-finden/museum/87-odenwald

87 is the propertyID

„Odenwald“ ist the value from my first customfield „Region“

Von: Scippy notifications@github.com Gesendet: Mittwoch, 30. September 2020 21:24 An: sachatrauwaen/OpenUrlRewriter OpenUrlRewriter@noreply.github.com Cc: GerritGer gerrit.stoever@online.de; Mention mention@noreply.github.com Betreff: Re: [sachatrauwaen/OpenUrlRewriter] Any chance to make it compatible with latest DNN 9.6.2 (#22)

To enable OpenUrlRewrite on PA you must disable "Do Redirect" in PA SEO settings, then clear cache. https://user-images.githubusercontent.com/16607338/94730221-2e65c800-0363-11eb-8ebb-195801eacaa0.jpg

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sachatrauwaen/OpenUrlRewriter/issues/22#issuecomment-701594470 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ3R3UDJBFNF3WD4IYV2ZPLSIOAVXANCNFSM4PWWXI2A . https://github.com/notifications/beacon/AQ3R3UAGI4OGUL5L2KMGDVTSIOAVXA5CNFSM4PWWXI2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFHIXWZQ.gif

Scippy commented 3 years ago

It is right. The Property ID is used for duplicated title. You can personalize page title working around view property layout file

Screenshot_1

GerritGer commented 3 years ago

Hi,

changing the settings for the Page title doesn’t change the URl. I would like an URL without the Typefields. I tried several times with different SEO-Settings for „View Type title“ e.g. blank field, [Type:0] or [Type:-1] Everytime i get the full type-fields in the url. I have no idea what else i could try.

Gerrit

Von: Scippy notifications@github.com Gesendet: Donnerstag, 1. Oktober 2020 11:53 An: sachatrauwaen/OpenUrlRewriter OpenUrlRewriter@noreply.github.com Cc: GerritGer gerrit.stoever@online.de; Mention mention@noreply.github.com Betreff: Re: [sachatrauwaen/OpenUrlRewriter] Any chance to make it compatible with latest DNN 9.6.2 (#22)

It is right. The Property ID is used for duplicated title. You can personalize page title working around view property layout file

https://user-images.githubusercontent.com/16607338/94794708-8b519480-03dc-11eb-8133-73beaa59c592.jpg

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sachatrauwaen/OpenUrlRewriter/issues/22#issuecomment-702024262 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ3R3UBOWUOQNZH6MDPBTHDSIRGRBANCNFSM4PWWXI2A . https://github.com/notifications/beacon/AQ3R3UD2HLAZC6NKODT6TVLSIRGRBA5CNFSM4PWWXI2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFHMAURQ.gif

Scippy commented 3 years ago

Yes it is right. There are two different setting to do: page URL and page Title The page URL always contains full type path, you can personalize only last field through SEO Layout Settings Page Title instead, it is completely customizable in View Property Layout File

GerritGer commented 3 years ago

Thanks for the info, too bad, that there is no way to get rid of the type Path.

What about the ventrian news Arcticles? Do i need an url Provider to get a short URL like www.mysite.com/ArticleTitle? On a older screenshort for the Site setting of „Open Url Rewriter“ i saw an entry „NewsArticlesUrlRuleProvider“. Do i need this provider ? I could find it.

Von: Scippy notifications@github.com Gesendet: Freitag, 2. Oktober 2020 08:40 An: sachatrauwaen/OpenUrlRewriter OpenUrlRewriter@noreply.github.com Cc: GerritGer gerrit.stoever@online.de; Mention mention@noreply.github.com Betreff: Re: [sachatrauwaen/OpenUrlRewriter] Any chance to make it compatible with latest DNN 9.6.2 (#22)

Yes it is right. There are two different setting to do: page URL and page Title The page URL always contains full type path, you can personalize only last field through SEO Layout Settings Page Title instead, it is completely customizable in View Property Layout File

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sachatrauwaen/OpenUrlRewriter/issues/22#issuecomment-702556131 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ3R3UBWVP6KJTNA4ILSY3TSIVYUTANCNFSM4PWWXI2A . https://github.com/notifications/beacon/AQ3R3UCDWNQCFHHAKFGEAVTSIVYUTA5CNFSM4PWWXI2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFHQCPYY.gif

sachatrauwaen / OpenUrlRewriter

Any chance to make it compatible with latest DNN 9.6.2 #22