Make WordPress Core

Opened 5 years ago

Last modified 3 months ago

#51792 new defect (bug)

Add Home URL to New User emails

Reported by: ipstenu's profile Ipstenu Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Users Keywords: has-patch needs-refresh
Focuses: Cc:

Description

Back when we all had one website and we jolly well liked it, it was easy to know "Oh someone signed up at my site!" Now that people manage multiple sites, some with similar names, it can be confusing as to which site, exactly, did the new user join.

We can save headaches in this one cool trick ... add the Home URL to the emails.

Attachments (2)

51792.diff (966 bytes) - added by Ipstenu 5 years ago.
51792-2.diff (971 bytes) - added by Ipstenu 5 years ago.

Download all attachments as: .zip

Change History (7)

@Ipstenu
5 years ago

#1 @mukesh27
5 years ago

  • Keywords needs-refresh added

Can you please add missing slash in home_url() for consistency?

$homeurl = home_url();

Replace to

$homeurl = home_url( '/' );

@Ipstenu
5 years ago

#2 @Ipstenu
5 years ago

Done

#3 @davidbaumwald
5 years ago

  • Keywords needs-refresh removed

#4 @davidbaumwald
5 years ago

  • Keywords needs-testing added

Sorry for the noise. Adding a call for testing.

#5 @huzaifaalmesbah
3 months ago

  • Keywords needs-refresh added; needs-testing removed

I attempted to apply the latest patch (51792-2.diff) against current trunk (7.0-alpha), but it does not apply cleanly.

Patch results:

  • src/wp-includes/pluggable.php → hunk failed (.rej created)

Since the patch cannot be applied, testing is not possible at this time. Because of this, keeping needs-testing is not actionable until the patch is refreshed.

Removing needs-testing and adding needs-refresh.

Note: See TracTickets for help on using tickets.