- ๐ฏ๐ตJapan ptmkenny
@pankaj1390 Why did you remove the comment line from the patch in #20? Your code in #23 and #24 is identical but you removed the documentation comment with no explanation.
- @ptmkenny opened merge request.
- ๐ฏ๐ตJapan ptmkenny
I made an MR based on the patch in #20 with a few updates to the language (add a comma, remove an extraneous "the").
- Status changed to Needs work
over 1 year ago 5:13pm 13 February 2023 - ๐บ๐ธUnited States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request โ as a guide.
Moving to NW for the issue summary update requested in #9
Also as a bug this will need a test case.
Why this is not include in core because number of user using ngnix?
- ๐ฏ๐ตJapan ptmkenny
@pankaj1390 Please read the issue priority guidelines โ and do not change the priority without a valid reason.
Also bugfixes always go into the working development version of Drupal, which is currently 10.1.