#60466 closed enhancement (fixed)
Introduce a contextual help tab for plugin dependencies
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Help/About | Keywords: | has-patch needs-copy-review |
Focuses: | Cc: |
Attachments (2)
Change History (24)
#4
@
12 months ago
- Keywords has-patch added; needs-patch removed
@swissspidy , I have added a patch. Kindly check if it can make it to 6.5 RC1
#5
@
12 months ago
- Keywords needs-patch added; has-patch removed
@adarshposimyth I don't see any patch or pull request.
#6
@
11 months ago
@adarshposimyth could you please quickly add your patch in the ticket so we can see if it can be quickly committed before RC1 or not? Otherwise I'll punt it to milestone 6.6 at the top of the hour since we'll reach the commit freeze hour :)
#9
@
10 months ago
Now, basically this well be a new help option in the help tab... But where do we place it? Do we place it in the plugin section? In the install section AND/OR the plugin overview list?
#10
@
8 months ago
- Milestone changed from 6.6 to 6.7
- Type changed from defect (bug) to enhancement
Sorry, but this is an enhancement, so, I am moving it to the next milestone 😐
@costdev I am ready to collaborate on this if needed 🙂
This ticket was mentioned in PR #6806 on WordPress/wordpress-develop by DorZuberi.
8 months ago
#12
- Keywords has-patch added; needs-patch removed
Trac ticket: https://core.trac.wordpress.org/ticket/60466
This ticket was mentioned in Slack in #core-test by audrasjb. View the logs.
8 months ago
#15
follow-up:
↓ 17
@
8 months ago
In general looks good to me. This line needs attention from my point of view:
In case plugin activation link is disabled, this is due to missing required dependencies for the said plugin.
Shouldn't it be:
plugin's
and here:
this is due to missing required dependencies for the said plugin.
Maybe it would be better to change the said
to:
related
or specific
or mentioned
I'm not the native, so definitely experienced eyes are needed.
This ticket was mentioned in Slack in #core by dorzki. View the logs.
8 months ago
#17
in reply to:
↑ 15
@
8 months ago
Replying to Boniu91:
In general looks good to me. This line needs attention from my point of view:
In case plugin activation link is disabled, this is due to missing required dependencies for the said plugin.
Shouldn't it be:
plugin's
and here:
this is due to missing required dependencies for the said plugin.
Maybe it would be better to change thesaid
to:
related
orspecific
ormentioned
I'm not the native, so definitely experienced eyes are needed.
Thanks for the feedback, I wish someone would proof-read it and give me final copy so I can update my PR ;)
This ticket was mentioned in Slack in #core-test by ankit-k-gupta. View the logs.
5 months ago
#19
@
5 months ago
This ticket is assigned to WP 6.7, and we have approximately two weeks until the Beta 1 release. Any progress on this ticket would be greatly appreciated!
#20
@
5 months ago
Had a look at the PR and I have added some suggested changes for the text, let me know what you think !
@audrasjb commented on PR #6806:
5 months ago
#22
Thanks everyone! Committed in https://core.trac.wordpress.org/changeset/59094
Just checking in here since there hasn't been any progress here and we're approaching the RC phase. Do you think this is required and still feasible for 6.5? Any help needed?