Skip to content
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

feat(cpn): Extend Log Viewer x-axis time format to show milliseconds #4276

Merged
merged 1 commit into from
Nov 10, 2023

Conversation

mha1
Copy link
Contributor

@mha1 mha1 commented Nov 2, 2023

Summary of changes:

  • extends log viewer's x-axis time format to show milliseconds

image

@pfeerick pfeerick added the companion Related to the companion software label Nov 4, 2023
@pfeerick pfeerick added this to the 2.10 milestone Nov 4, 2023
@pfeerick pfeerick added enhancement ✨ New feature or request don't merge labels Nov 4, 2023
@pfeerick pfeerick changed the title CPN: extend Log Viewer's x-axis time format to also show milliseconds feat(cpn):Extend Log Viewer's x-axis time format to show milliseconds Nov 4, 2023
@pfeerick pfeerick changed the title feat(cpn):Extend Log Viewer's x-axis time format to show milliseconds feat(cpn): Extend Log Viewer's x-axis time format to show milliseconds Nov 4, 2023
@pfeerick pfeerick changed the title feat(cpn): Extend Log Viewer's x-axis time format to show milliseconds feat(cpn): Extend Log Viewer x-axis time format to show milliseconds Nov 4, 2023
@mha1
Copy link
Contributor Author

mha1 commented Nov 6, 2023

why the "don't merge" label?

@pfeerick
Copy link
Member

pfeerick commented Nov 6, 2023

Because you're touching companion... Don't you remember there is a no-touch policy in place atm? 😛

@mha1
Copy link
Contributor Author

mha1 commented Nov 6, 2023

Now that you mention it I vaguely remember Neil working on some stuff ...

@mha1 mha1 force-pushed the PR_CPN_logviewer_milliseconds branch from 121cc49 to 066cb29 Compare November 6, 2023 11:19
@elecpower
Copy link
Collaborator

Sorry Michael but some bad news in that #4270 breaks this one. qCustomplot upgrade required changes here

@mha1
Copy link
Contributor Author

mha1 commented Nov 6, 2023

Don't worry, I'll wait for your PR to be merged and try again

@pfeerick
Copy link
Member

pfeerick commented Nov 10, 2023

@mha1 You should be right to rebase and/or try again

@mha1 mha1 force-pushed the PR_CPN_logviewer_milliseconds branch from 066cb29 to e74b0b5 Compare November 10, 2023 08:22
@mha1
Copy link
Contributor Author

mha1 commented Nov 10, 2023

@mha1 You should be right to rebase and/or try again

rebase done

@pfeerick pfeerick merged commit da0950c into EdgeTX:main Nov 10, 2023
3 checks passed
@pfeerick pfeerick deleted the PR_CPN_logviewer_milliseconds branch November 10, 2023 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
companion Related to the companion software enhancement ✨ New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants