Making WordPress.org

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#489 closed enhancement (fixed)

Adding text to Rosetta contact forms

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: iandunn's profile iandunn
Milestone: Priority: normal
Component: International Sites (Rosetta) Keywords: has-patch commit
Cc:

Description

I get a lot of emails via http://ru.wordpress.org/contact/. Most of them are lost password requests, some are copyright infringement reports, and the rest are other support requests.

See also the thread on Polyglots, which suggests introducing an option to disable the contact form.

I would like to add some text to the form with links to FAQ pages in Codex (the same pages we generally link to in forum replies).

When I open the page for editing, I see this as the content:

The contents of this page are filled automatically, you don't need to spend time here. Just change the title.

There should be a way to replace this with a text that would be displayed on the front-end.

Attachments (1)

meta-489.patch (1.3 KB) - added by SergeyBiryukov 10 years ago.

Download all attachments as: .zip

Change History (9)

#1 @SergeyBiryukov
10 years ago

  • Type changed from defect to enhancement

This ticket was mentioned in IRC in #wordpress-meta by sams. View the logs.


10 years ago

#3 @iandunn
10 years ago

  • Owner set to iandunn
  • Resolution set to fixed
  • Status changed from new to closed

In 704:

Rosetta theme: Allow admins to customize the Contact form introduction.

If they change the page contents, the new contents will be displayed instead of the default introduction.
Fixes #489

#4 @iandunn
10 years ago

Since all of the sites currently have the "The contents of this page are filled automatically, you don't need to spend time here. Just change the title." text in post_content, just adding the_content() to the template was undesireable, because then all the sites would be displaying that message until each one was manually changed (which will probably never happen for many sites).

So, I thought the simplest transition would be to display the_content() if the admin had changed it from the original message, but otherwise leave the default introduction.

So, admins can now replace the current introduction by changing the post content to be whatever they'd like.

#5 @SergeyBiryukov
10 years ago

This works great, thanks.

#6 @SergeyBiryukov
10 years ago

  • Keywords has-patch commit added
  • Resolution fixed deleted
  • Status changed from closed to reopened

Apparently some sites had an additional sentence in the default content. Some examples:

They all currently display default content instead of a proper introduction:

Please, neither delete this page, nor change its slug!

The contents of this page are filled automatically, you don't need to spend time here. Just change the title.

I'd suggest using strpos() instead of string comparison, see meta-489.patch.

#7 @iandunn
10 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 720:

Rosetta theme: Make the detection of default content less strict.

There are a few sites with variants on the default contact form introduction (e.g.,
http://en-ca.wordpress.org/contact/) and the previous method was too strict to include them.

props SergeyBiryukov
fixes #489

#8 @iandunn
10 years ago

Thanks for catching that :)

Note: See TracTickets for help on using tickets.