Show saved log recorders#23858
Conversation
daniel-beck
left a comment
There was a problem hiding this comment.
Works in manual testing, both with and without the UI experiment.
(IMO: This particular pattern seems like something that should be solved differently, before it proliferates into plugins.)
|
/label ready-for-merge This PR is now ready for merge, after ~24 hours, we will merge it if there's no negative feedback. Thanks! |
Discussed with Jan, we've got an idea for improving it. |
|
Nice, that aligns with |
* Show saved log recorders * Use pattern from rest of settings rework PR
Fixes #23856
Testing done
Created a log recorder and it now shows up again
Proposed changelog entries
Proposed changelog category
/label
Proposed upgrade guidelines
N/A
Submitter checklist
@Restrictedor have@since TODOJavadocs, as appropriate.@Deprecated(since = "TODO")or@Deprecated(forRemoval = true, since = "TODO"), if applicable.evalto ease future introduction of Content Security Policy (CSP) directives (see documentation).Desired reviewers
N/A
Before the changes are marked as
ready-for-merge:Maintainer checklist
upgrade-guide-neededlabel is set and there is a Proposed upgrade guidelines section in the pull request title (see example).lts-candidateto be considered.