- Issue created by @joachim
- First commit to issue fork.
- 🇫🇷France vargenty
vargenty → changed the visibility of the branch 3506386-incompleteinaccurate-docs-for to hidden.
- 🇫🇷France vargenty
vargenty → changed the visibility of the branch 3506386-incompleteinaccurate-docs-for to active.
- 🇫🇷France vargenty
Hello, I'm working on it. It's my first contribution !
- Merge request !11202Issue #3506386: incomplete/inaccurate docs for getCountNewComments::getCountNewComments() → (Closed) created by Unnamed author
- 🇫🇷France vargenty
Hello, here is my first contribution on Drupal. I hope i'm doing good !
- 🇬🇧United Kingdom joachim
Looks good, though the first line of the doc block is now too long. We need to shorten it. I'd say the word 'available' isn't adding any meaning, so we can remove that.
Remember to set an issue to 'needs review' when it's ready.
(I somehow managed to copy-paste the interface name wrong though...)
- 🇫🇷France vargenty
Thank you for your comment. I fixed the first line as you adviced.
Sorry for the status, I couldn't change it after my MR, I am not sure why. - 🇳🇿New Zealand quietone
Yes, this reads better, thanks. There is just one more small thing, which I commented on in the MR. For background on that see 📌 [Meta] Use Install/Uninstall consistently for turning modules/themes on/off (not Enable) Fixed .
- First commit to issue fork.
- 🇮🇳India adwivedi008
Hello @quietone
I have added the recommended changes
Please review and suggest any other changes required - 🇺🇸United States smustgrave
Feedback from @quietone appears to be addressed
Automatically closed - issue fixed for 2 weeks with no activity.