Opened 6 months ago
Last modified 6 weeks ago
#60466 assigned enhancement
Introduce a contextual help tab for plugin dependencies
Reported by: | desrosj | Owned by: | costdev |
---|---|---|---|
Milestone: | 6.7 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Help/About | Keywords: | has-patch needs-copy-review |
Focuses: | Cc: |
Attachments (2)
Change History (19)
#4
@
5 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
@
5 months ago
- Keywords needs-patch added; has-patch removed
@adarshposimyth I don't see any patch or pull request.
#6
@
5 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
@
3 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
@
7 weeks 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.
6 weeks 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.
6 weeks ago
#15
follow-up:
↓ 17
@
6 weeks 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.
6 weeks ago
#17
in reply to:
↑ 15
@
6 weeks 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 ;)
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?