Make WordPress Core

Opened 12 years ago

Closed 10 years ago

#21089 closed enhancement (fixed)

Add admin url to install notification email

Reported by: georgestephanis's profile georgestephanis Owned by: wonderboymusic's profile wonderboymusic
Milestone: 4.1 Priority: normal
Severity: minor Version: 3.4
Component: Text Changes Keywords: has-patch ux-feedback
Focuses: Cc:

Description

I've seen a number of individuals inquiring as to the login url of their WordPress site for administration purposes. Let's give it to them by changing:

Your new WordPress site has been successfully set up at:

http://wordpress.url

You can log in to the administrator account with the following information:

Username: admin
Password: The password you chose during the install.

We hope you enjoy your new site. Thanks!

--The WordPress Team
http://wordpress.org/

to

Your new WordPress site has been successfully set up at:

http://wordpress.url

You can log in to the administrator account with the following information:

Username: admin
Password: The password you chose during the install.
Location: http://wordpress.url/wp-admin

We hope you enjoy your new site. Thanks!

--The WordPress Team
http://wordpress.org/

Attachments (2)

21089.diff (1.3 KB) - added by kobenland 12 years ago.
Streamlined welcome messages
21089.2.patch (2.1 KB) - added by georgestephanis 12 years ago.

Download all attachments as: .zip

Change History (14)

#1 @sabreuse
12 years ago

Completely in favor of this. For wording, I'd rather put the location first in this list, just because I think the order fits the logical/reading flow a bit better (go HERE, then type THIS, then what-did-I-do-with-that-password-oh-noes).

edit:

New site emails in multisite do include a login link, with the wording "Log in here:" rather than "Location:" If we do this, let's make it so both messages get the same wording & layout.

Last edited 12 years ago by sabreuse (previous) (diff)

#2 @jane
12 years ago

I would put the url before the u/p and on its own line. Otherwise, good by me.

#3 @georgestephanis
12 years ago

Should the link go to wp-login.php or wp-admin ?

@kobenland
12 years ago

Streamlined welcome messages

#4 @georgestephanis
12 years ago

  • Keywords has-patch ux-feedback added; needs-patch 2nd-opinion removed

I was thinking something closer to 21089.2.patch -- we already have wp_login_url() -- may as well use it for the install notification.

I wouldn't mind some later enhancement that lets you pass an argument to wp_login_url() to specify the blogID you want the login URL for, for multisite installs.

#5 @iseulde
11 years ago

  • Component changed from General to Text Changes

#6 @wonderboymusic
10 years ago

  • Owner set to wonderboymusic
  • Status changed from new to reviewing

#7 @SergeyBiryukov
10 years ago

  • Milestone changed from Awaiting Review to 4.1
  • Owner changed from wonderboymusic to SergeyBiryukov
  • Status changed from reviewing to accepted

#8 @SergeyBiryukov
10 years ago

  • Owner changed from SergeyBiryukov to wonderboymusic
  • Status changed from accepted to assigned

Oops.

#9 @wonderboymusic
10 years ago

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

In 29806:

In the wp_new_blog_notification() email template, include the login url.

Props obenland, georgestephanis.
Fixes #21089.

#10 @SergeyBiryukov
10 years ago

In 29807:

Reorder placeholders for consistency. see [29806].

see #21089.

#11 @nacin
10 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Is there a reason why "Log in here:" was moved from the bottom to the top, in the place where it already existed? This broke a string and it also seems to hurt the flow a bit. It should be: You log in with this information / username / password / and you log in at this url.

#12 @nacin
10 years ago

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

In 30742:

Adjust [29806] and revert [29807] to restore placeholder and email copy order.

Avoids unnecessarily changing a (large, carefully formatted) translated string from 4.0.

fixes #21089.

Note: See TracTickets for help on using tickets.