- Issue created by @joachim
- First commit to issue fork.
- Merge request !8772Issue #3445909 by seanB, smustgrave, alexpott, catch: Add static caching to... → (Closed) created by VinmayiSwamy
- 🇮🇳India VinmayiSwamy
VinmayiSwamy → changed the visibility of the branch 3461421-getprocessplugins-normalises-the to hidden.
- Merge request !87733461421:Fix getProcessPlugins() to avoid normalizing the process array twice → (Closed) created by VinmayiSwamy
- Status changed to Needs review
about 2 months ago 2:54pm 15 July 2024 - 🇮🇳India VinmayiSwamy
Hey @joachim
Thanks for reviewing the MR. I added else{} to the if{} at the top of the method as suggested. Kindly please review.
Thanks!
- First commit to issue fork.
- 🇮🇳India ankitv18
@VinmayiSwamy I have updated the MR!8773 with single line of code.
cc: @joachim
- Status changed to RTBC
about 2 months ago 6:10pm 15 July 2024 - 🇺🇸United States smustgrave
Fix seems straight forward. Though just FYI the issue was created just today and tagged novice for new users.
@ankitv18 based on your post history believe you definitely can probably avoid novice tagged issues
@vinmayiswamy looking at your post history believe you've also probably hit the line where you can probably start going forward on non novice issues.
Now if an issue is tagged for novice and sits around for say a month then it's fair game but that wasn't the case here.
Thanks
- Status changed to Needs review
about 2 months ago 11:49pm 15 July 2024 - 🇳🇿New Zealand quietone New Zealand
There are code changes here that were made after the RTBC. Setting back to NR
- Status changed to RTBC
about 2 months ago 9:04am 16 July 2024 - 🇬🇧United Kingdom catch
Committed/pushed to 11.x and cherry-picked back through to 10.3.x, thanks!
- Status changed to Fixed
about 2 months ago 2:36pm 19 July 2024 Automatically closed - issue fixed for 2 weeks with no activity.