Open bowenliang123 opened 2 months ago
Attention: Patch coverage is 0%
with 45 lines
in your changes missing coverage. Please review.
Project coverage is 0.00%. Comparing base (
8056235
) to head (1c1d58d
). Report is 8 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
listing configurations should be restricted to admin only. additionally, we may need to have another parameter to define the display mode for configuration values, e.g.
PS: I remember there was a similar request to have a configuration display mode of configurations for retrieving batch info
:mag: Description
Issue References ๐
This pull request fixes #
Describe Your Solution ๐ง
list config
in admin cli tool , for listing all the server configs, which is useful for checking runtime configs when before/after refreshing the configs of user-defaults or non-user-defaultsTypes of changes :bookmark:
Test Plan ๐งช
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests
Checklist ๐
Be nice. Be informative.