#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
@
22 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
@
22 months ago
- Keywords needs-patch added; has-patch removed
@adarshposimyth I don't see any patch or pull request.
#6
@
22 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
@
20 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
@
18 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.
18 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.
18 months ago
#15
follow-up:
โย 17
@
18 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.
18 months ago
#17
in reply to:
โย 15
@
18 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 thesaidto:
relatedorspecificormentioned
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.
15 months ago
#19
@
15 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
@
15 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:
15 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?