WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 months ago

#21089 closed enhancement (fixed)

Add admin url to install notification email

Reported by: georgestephanis Owned by: 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 3 years ago.
Streamlined welcome messages
21089.2.patch (2.1 KB) - added by georgestephanis 3 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 @sabreuse3 years ago

Completely in favor of this. For wording, I'd rather put the location first in the 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).

Version 0, edited 3 years ago by sabreuse (next)

comment:2 @jane3 years ago

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

comment:3 @georgestephanis3 years ago

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

@kobenland3 years ago

Streamlined welcome messages

@georgestephanis3 years ago

comment:4 @georgestephanis3 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.

comment:5 @iseulde19 months ago

  • Component changed from General to Text Changes

comment:6 @wonderboymusic5 months ago

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

comment:7 @SergeyBiryukov5 months ago

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

comment:8 @SergeyBiryukov5 months ago

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

Oops.

comment:9 @wonderboymusic5 months 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.

comment:10 @SergeyBiryukov5 months ago

In 29807:

Reorder placeholders for consistency. see [29806].

see #21089.

comment:11 @nacin3 months 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.

comment:12 @nacin3 months 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.