WordPress.org

Make WordPress Core

Opened 3 weeks ago

Closed 9 days ago

Last modified 9 days ago

#51489 closed task (blessed) (fixed)

Application Passwords: Add some additional explanation for Multisite instances

Reported by: georgestephanis Owned by: sarahricker
Milestone: 5.6 Priority: normal
Severity: normal Version: trunk
Component: Login and Registration Keywords: has-patch needs-copy-review needs-refresh
Focuses: multisite, rest-api Cc:

Description

To properly set expectations for users, when the WordPress install is a Multisite install, and the current user has roles on multiple blogs, display an extra blurb on the authorization flow and application passwords section of the user profile screen saying something to the effect of:

Application Passwords will also grant the application access the other {$num_blogs} blogs that you have permissions on.

(along with possibly listing them if it's a short list, or a link to a page in the user dashboard or multisite dashboard listing the user's blogs)

As we'd be displaying it in two places, I'm unsure if it would be worth abstracting to a function or just inlining it both places.

For future v2 buildout, as we're looking at adding additional ability to scope application passwords access, being able to limit the sites they are used on may be worth including in that addition.

Props @spacedmonkey for asking multisite related questions that made me think of this.

Change History (22)

This ticket was mentioned in Slack in #core-passwords by georgestephanis. View the logs.


3 weeks ago

#2 @johnjamesjacoby
3 weeks ago

I support additional clarity here. Suggest to start by linking to the “My Sites” admin page, and iterate if it is determined to be insufficient.

This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.


12 days ago

#4 @hellofromTonya
12 days ago

  • Type changed from enhancement to task (blessed)

As discussed in scrub, converting this to task.

#5 @sarahricker
12 days ago

Per today's bug scrub, this ticket (now a task) will stay in 5.6 (and after Beta 1 if necessary) as it aims to refine a new feature that has already been merged into trunk.

@georgestephanis and @spacedmonkey - Can you add the exact copy for each location it will be added to this ticket to ensure it is good to go?

Thanks!

This ticket was mentioned in PR #635 on WordPress/wordpress-develop by georgestephanis.


11 days ago

  • Keywords has-patch added

Add explanation text in multisite environments when the user has multiple blogs clarifying that application passwords will grant access to all their sites.

Trac ticket: https://core.trac.wordpress.org/ticket/51489

#7 @georgestephanis
11 days ago

@sarahricker Created a github pr for review and discussion. Honestly I'm horrid on verbiage and phrasing, but the context there should be enough for someone better with words for users to make it intelligible. :\

This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.


11 days ago

#9 @sarahricker
11 days ago

  • Keywords needs-copy-review added

Thanks @georgestephanis, that is perfect! Will take a look. Will try to get it in Beta 1, but if not - Beta 2!

#10 @TimothyBlynJacobs
11 days ago

This will grant access to all 10 blogs in this installation that you have permissions on. and Application passwords grant access to all 10 blogs in this installation that you have permissions on.

#11 @sarahricker
11 days ago

  • Owner set to sarahricker
  • Status changed from new to accepted

#12 @prbot
11 days ago

TimothyBJacobs commented on PR #635:

@georgestephanis looks like there are linting failures.

#13 @SergeyBiryukov
11 days ago

  • Keywords needs-refresh added

A couple of notes:

  • The strings should use _n() instead of __() here, many languages have more than one plural form.
  • sizeof() should be replaced with count(), see [49193].

#14 @prbot
11 days ago

georgestephanis commented on PR #635:

Yeah, just fixed.

#15 @prbot
11 days ago

TimothyBJacobs commented on PR #635:

@SergeyBiryukov's also left comments on the ticket.

#16 @georgestephanis
11 days ago

Updated addressing _n and count feedback.

This ticket was mentioned in Slack in #core-passwords by georgestephanis. View the logs.


10 days ago

#18 @TimothyBlynJacobs
10 days ago

  • Component changed from Networks and Sites to Login and Registration
  • Focuses rest-api added

#19 @TimothyBlynJacobs
9 days ago

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

In 49270:

App Passwords: Explain that apps can access to all sites in a network.

Props georgestephanis, spacedmonkey, johnjamesjacoby, SergeyBiryukov, marybaum.
Fixes #51489.

#20 @johnjamesjacoby
9 days ago

Is there a compelling reason why the HTML for this feature does not follow the pattern set by every other user profile feature on this page?

This feature would feel much more complete if it did not look as unique and unfamiliar.

Any objections to reopening & continuing to iterate on this during beta?

#21 @georgestephanis
9 days ago

@johnjamesjacoby Which aspect of the html do you mean?

And I'm fine with reopening this ticket if it's about the new multisite line specifically, but if it's a grander rearchitecting, may be better to do a new ticket.

#22 @SergeyBiryukov
9 days ago

In 49282:

Coding Standards: Update some alignment in wp-admin/authorize-application.php for consistency.

Save blogs count to a variable for reusing.

Follow-up to [49270].

See #51489.

Note: See TracTickets for help on using tickets.