similar topic: http://drupal.org/node/494462#new

no reason to block it. it should visitable on search engine and let users can find us.

#52 teezee

I mean, the contact module is core, but on most sites something like a contact section (contact press department, contact webmasters) with important address information will not be indexed if u would have aliases like /contact/press or /contact/webmaster.

Is it really up to Drupal core to enforce /contact/ as a disallowed path just because core contains a module that uses /contact ?

CommentFileSizeAuthor
#16 905576-14-remove-contact-from-robots-D6.patch497 bytesidflood
#15 905576-15-remove-contact-from-robots-D6.patch683 bytestim.plunkett
#12 contact.patch497 bytesdroplet
#9 905576-9-remove-contact-from-robots.patch497 bytesidflood
#6 905576-6-remove-contact-from-robots.patch683 bytestim.plunkett
#3 905576-3-remove-contact-from-robots.patch512 bytesidflood

Comments

teezee’s picture

teezee commented

Subscribe, thanks @droplet for posting :)

abaddon’s picture

abaddon commented

not to mention the people who use google as the address bar or if the contact link isnt that visible and they search for "site:example.com contact"..

idflood’s picture

idflood commented
Status: Active » Needs review
StatusFileSize
new 905576-3-remove-contact-from-robots.patch512 bytes

Here is a quick patch. Contact page must be crawled by search engine, it's definitly one of the most important page to be referenced for most of the sites I do.

droplet’s picture

droplet commented
Status: Needs review » Reviewed & tested by the community
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Version: 7.x-dev » 6.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Weird. I've no idea why that would be blocked.

Committed to HEAD. Marking down to 6.x.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new 905576-6-remove-contact-from-robots.patch683 bytes

Easy peasy.

droplet’s picture

droplet commented
Status: Needs review » Reviewed & tested by the community

just started up my box to patch it and you did.. haha
patch works

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 905576-6-remove-contact-from-robots.patch, failed testing.

idflood’s picture

idflood commented
StatusFileSize
new 905576-9-remove-contact-from-robots.patch497 bytes

Maybe this one will pass.

idflood’s picture

idflood commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 905576-9-remove-contact-from-robots.patch, failed testing.

droplet’s picture

droplet commented
Status: Needs work » Needs review
StatusFileSize
new contact.patch497 bytes

try this

Status: Needs review » Needs work

The last submitted patch, contact.patch, failed testing.

droplet’s picture

droplet commented

ok. "testbot work ONLY with D7 patches"

http://drupal.org/node/894166

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new 905576-15-remove-contact-from-robots-D6.patch683 bytes

Oh, duh. Back to RTBC per #7.

idflood’s picture

idflood commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 905576-14-remove-contact-from-robots-D6.patch497 bytes

Ok, I forgot that the tests only work for d7 ( http://drupal.org/node/894166#comment-3693940 ). So here is the same patch as #9, only filename is changed to prevent the patch from beeing incorrectly tested.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Needs review » Reviewed & tested by the community

cross-post

izmeez’s picture

izmeez commented

subscribing

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed and pushed.

Status: Fixed » Closed (fixed)
Issue tags: -robots.txt

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +robots.txt

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.