-
Notifications
You must be signed in to change notification settings - Fork 206
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Client log view improvements #1385
Conversation
The GOAT returns |
This reminds me - we should open a ticket to grab log messages from before the UI connects. We do this in a hacky way right now, but only for the first client. |
This needs to be done in the composition Vue API for the transition to Vue3, I already have Spacing, coloring, reactivity, and styling on the vue3 branch. Will PR soon, we should merge this either directly in that PR or after we merge that one. |
{{ getLogLevelFromIndex(level) }} | ||
</v-col> | ||
<v-col cols="6" md="4"> | ||
<v-switch v-model="selectedLogLevels[level]" dark color="#ffd843" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whats the reason to replace the button toggle with switches?
closes #1384 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it work? I have no idea! Sure Lgtm tho
@DevonRD not using pnpm anymore until we can bump to Vue3 because of legacy peer dependency resolution issues
These changes will likely have to be looked at again during the Vue3 changes |
Fixes the following issues with client log view:
Adds the following features to log view:
*A couple files have inconsequential changes from linting/formatting