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

ui: Device to its own file #32987

Closed
wants to merge 7 commits into from

Conversation

sunnyhaibin
Copy link
Contributor

Description

Split Device class into its own file under qt/device directory. This keeps functions readable and organized.

Similar to #32059

@github-actions github-actions bot added the ui label Jul 15, 2024
Copy link
Contributor

github-actions bot commented Jul 15, 2024

Thanks for contributing to openpilot! In order for us to review your PR as quickly as possible, check the following:

  • Convert your PR to a draft unless it's ready to review
  • Read the contributing docs
  • Before marking as "ready for review", ensure:
    • the goal is clearly stated in the description
    • all the tests are passing
    • the change is something we merge
    • include a route or your device' dongle ID if relevant

@sunnyhaibin sunnyhaibin changed the title UI split device ui: split Device to its own file Jul 15, 2024
@sunnyhaibin sunnyhaibin changed the title ui: split Device to its own file ui: Device to its own file Jul 15, 2024
Copy link
Contributor

github-actions bot commented Aug 3, 2024

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

@github-actions github-actions bot added the stale label Aug 3, 2024
Copy link
Contributor

github-actions bot commented Aug 4, 2024

UI Screenshots

@github-actions github-actions bot removed the stale label Aug 4, 2024
Copy link
Contributor

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

@github-actions github-actions bot added the stale label Aug 20, 2024
@sunnyhaibin
Copy link
Contributor Author

Closing as it's deemed as Negative expected value.

@sunnyhaibin sunnyhaibin deleted the ui-split-device branch August 21, 2024 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants