? ? Success

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
19 Oct 2016

I see lots of confusion about the "preferred domain" setting in the system sef plugin. I hope this PR will help to reduce that. Please note that one "improvement" in this tooltip is the explanation that https://example.com and https://www.example.com are different domains

I used this as a reference https://support.google.com/webmasters/answer/44231?hl=en I hope I haven't misunderstood anything about this param

2ta2 1

avatar brianteeman brianteeman - open - 19 Oct 2016
avatar andrepereiradasilva
andrepereiradasilva - comment - 19 Oct 2016

well, imho, if you have https://example.com and https://www.example.com you should be making HTTP 301 redirects to your prefered domain via web server, not adding canonical domains.

avatar brianteeman
brianteeman - comment - 19 Oct 2016

@andrepereiradasilva I agree with you about that but for the purpose of how this plugin works it should be noted that they are separate domains

avatar andrepereiradasilva
andrepereiradasilva - comment - 19 Oct 2016

@andrepereiradasilva I agree with you about that but for the purpose of how this plugin works it should be noted that they are separate domains
Show all checks

Yes i understand, but i really think you should use different domains as example not www.example.com and his naked domain version. Because if you are doing this, imho you are not doing it right ?

avatar brianteeman
brianteeman - comment - 19 Oct 2016

I was trying to show that they are different domains as well and not just
the obvious a.com and b.com

On 19 October 2016 at 14:28, andrepereiradasilva notifications@github.com
wrote:

@andrepereiradasilva https://github.com/andrepereiradasilva I agree
with you about that but for the purpose of how this plugin works it should
be noted that they are separate domains
Show all checks

Yes i understand, but i really think you should use different domains as
example not www.example.com and his naked domain. Because if you are
doing this, imho you are not doing it right ?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#12472 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8VXjHLvdUOWHurxvs88Ag_0O-JQrks5q1hsbgaJpZM4Ka64a
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar infograf768
infograf768 - comment - 19 Oct 2016

agree with andre. this is confusing. please use different domains

avatar brianteeman
brianteeman - comment - 19 Oct 2016

It is deliberate to show that they are different domains - sorry but I am
not going to change that - so it either gets accepted or rejected but I am
not changing it

On 19 October 2016 at 16:20, infograf768 notifications@github.com wrote:

agree with andre. this is confusing. please use different domains


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#12472 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8dRYCmonb2ZI2y-BoslVHi_-OvGgks5q1jU3gaJpZM4Ka64a
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar mbabker
mbabker - comment - 19 Oct 2016

Brian's right here. It needs to be clear that example.com and www.example.com, though most DNS configurations by default alias it to act in this way, are NOT one in the same. And yes users need to be aware of SEO implications of not standardizing on one and redirecting the other, but that part's outside the scope of what can be done in Joomla.

As is, this tooltip reads fine to me.

avatar jeckodevelopment jeckodevelopment - change - 19 Oct 2016
Labels Added: ? ?
avatar jeckodevelopment jeckodevelopment - test_item - 19 Oct 2016 - Tested successfully
avatar jeckodevelopment
jeckodevelopment - comment - 19 Oct 2016

I have tested this item successfully on f005352


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12472.

avatar brianteeman brianteeman - change - 23 Oct 2016
Status New Pending
avatar brianteeman brianteeman - change - 23 Oct 2016
Category Language & Strings Plugins
avatar brianteeman brianteeman - change - 29 Oct 2016
Labels Added: ?
Removed: ?
avatar conconnl conconnl - test_item - 4 Nov 2016 - Tested successfully
avatar conconnl
conconnl - comment - 4 Nov 2016

I have tested this item successfully on f005352

Totally agree with this patch.

Tested successfully at PBF NL


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12472.

avatar jeckodevelopment jeckodevelopment - change - 4 Nov 2016
The description was changed
Status Pending Ready to Commit
Labels
avatar jeckodevelopment jeckodevelopment - change - 4 Nov 2016
Labels Added: ?
avatar jeckodevelopment jeckodevelopment - edited - 4 Nov 2016
avatar jeckodevelopment
jeckodevelopment - comment - 4 Nov 2016

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12472.

avatar joomla-cms-bot joomla-cms-bot - change - 4 Nov 2016
Category Language & Strings Plugins Administration Language & Strings
avatar jeckodevelopment jeckodevelopment - change - 4 Nov 2016
Labels Removed: ?
avatar jeckodevelopment jeckodevelopment - change - 4 Nov 2016
Milestone Added:
avatar rdeutz rdeutz - reference | 4e7e05b - 4 Nov 16
avatar rdeutz rdeutz - merge - 4 Nov 2016
avatar rdeutz rdeutz - change - 4 Nov 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-11-04 13:05:29
Closed_By rdeutz
avatar rdeutz rdeutz - close - 4 Nov 2016
avatar rdeutz rdeutz - merge - 4 Nov 2016
avatar rdeutz rdeutz - close - 4 Nov 2016
avatar brianteeman
brianteeman - comment - 4 Nov 2016

Thanks

avatar brianteeman brianteeman - head_ref_deleted - 4 Nov 2016

Add a Comment

Login with GitHub to post a comment