- πΊπΈ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.
This could use an issue summary update using the default template for steps to reproduce, proposed solution, and remaining tasks. To help the committers later.
- Status changed to Needs review
11 months ago 5:45pm 18 January 2024 - last update
11 months ago Build Successful - Status changed to Needs work
11 months ago 6:05pm 18 January 2024 The Needs Review Queue Bot β tested this issue.
While you are making the above changes, we recommend that you convert this patch to a merge request β . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)
- Merge request !6232Issue #2199001: Statically cache node access grants β (Open) created by Liam Morland
- Status changed to Needs review
11 months ago 7:02pm 18 January 2024 - π©πͺGermany Anybody Porta Westfalica
Thanks for the reroll @Liam Morland! Code LGTM! Could you perhaps provide it as MR so the tests can run and we can hopefully finish this soon?
Still has a "Needs issue summary update" tag.
- π©πͺGermany Anybody Porta Westfalica
Sorry @Liam Morland shame on me, I didn't see #40 just #38 -.- Thank you!
- Status changed to Needs work
11 months ago 2:32pm 25 January 2024 - πΊπΈUnited States smustgrave
Have not tested but see it was previously tagged for issue summary which still appears to be needed. Recommend using standard issue template even if the section doesn't apply, leaving N/A helps reviewers more then it missing.