Problem/Motivation
The initiatives responsibilities link is incorrect, it links to community initiatives rather than strategic ones and either way neither of those pages list the responsibilities of the initiative leads
Proposed resolution
Fix the link.
Remaining tasks
Review.
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | 3196388-12.png | 102.24 KB | kleiton_rodrigues |
| #11 | Screenshot 2021-03-08 at 11.42.10.png | 41.92 KB | gauravvvv |
| #10 | post-patch.png | 56.29 KB | guilhermevp |
| #8 | 3196388-8.patch | 574 bytes | ranjith_kumar_k_u |
| #6 | 3196388-6.patch | 607 bytes | Pooja Ganjage |
Comments
Comment #2
gábor hojtsyComment #3
gábor hojtsyComment #4
stephencamilo commentedI've read the patch and it seems fine.
Comment #5
catchComment #6
Pooja Ganjage commentedHi,
Creating a patch for this issue as patch got failed.
Please review the patch.
Thanks.
Comment #7
Pooja Ganjage commentedComment #8
ranjith_kumar_k_u commentedUpdated
Comment #9
guilhermevp commentedPatch updates documentation acordigly!
Comment #10
guilhermevp commentedAdded image evidence.
Comment #11
gauravvvv commentedPatch #8, Fixed the link for initiative responsibilities.
RTBC +1
Comment #12
kleiton_rodrigues commentedFixed the link for The Drupal core initiative coordinators oversee approved projects.
RTBC +1
Comment #13
alexpottCommitted and pushed 5c3ac0a573 to 9.2.x and 0d1d758a9c to 9.1.x. Thanks!
@guilhermevp, @Gauravmahlawat, @kleiton_rodrigues Thank you for looking into this issue.
Posting screenshots of your codebase or CLI does not advance the issue, since the automated testing infrastructure tells us whether the patch applies correctly.
So, I've removed the issue credit for that screenshot. In the future, you can get credit for issues by reading the issue to understand its purpose, and posting your review or testing of that purpose. With issues like this getting credit for a review is hard unless your review has a material impact on the patch. Thank you!
See the issue credit guidelines for more information.
Comment #16
gauravvvv commentedHi @alexpott, I haven't posted the screenshot for CLI. I posted the screenshot of the URL which was changed by the code. No other way to review the issue then this.