- Issue created by @vensires
- First commit to issue fork.
- First commit to issue fork.
- Status changed to Needs review
7 months ago 11:45am 17 April 2024 - Status changed to Needs work
7 months ago 1:49pm 17 April 2024 - Status changed to Needs review
7 months ago 3:51pm 17 April 2024 - Status changed to Needs work
7 months ago 4:53pm 17 April 2024 The Needs Review Queue Bot โ tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide โ to find step-by-step guides for working with issues.
- Status changed to RTBC
7 months ago 5:25pm 17 April 2024 - Status changed to Needs work
7 months ago 11:06pm 17 April 2024 - ๐ฌ๐งUnited Kingdom alexpott ๐ช๐บ๐
Added a comment to the MR about the use of the authenticated role.
- ๐ฎ๐ณIndia pradhumanjainOSL
pradhumanjain2311 โ made their first commit to this issueโs fork.
- ๐ฉ๐ชGermany SolimanHarkas
@pradhumanjain2311 With all my respect, I must point out that simply copying the suggested code and adding it, without local testing or noticing its duplication, isn't an appropriate approach to contributing to Drupal.
- Status changed to Needs review
7 months ago 7:14pm 18 April 2024 - Status changed to RTBC
7 months ago 7:29pm 18 April 2024 - Status changed to Fixed
7 months ago 9:41am 19 April 2024 - ๐ฌ๐งUnited Kingdom catch
Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks!
Automatically closed - issue fixed for 2 weeks with no activity.