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

Merge summer changes #7

Merged
merged 52 commits into from
Sep 27, 2024
Merged

Merge summer changes #7

merged 52 commits into from
Sep 27, 2024

Conversation

michael-m-2983
Copy link
Member

@michael-m-2983 michael-m-2983 commented Sep 1, 2024

The gist of this PR is that the entire application has been rewritten, and it now works in a different way.

Frontend

Scouters can use the turbo scout frontend, which may be hosted on the turbo scout website as it was previously. It also now has the eventual capability to be distributed as a single html file, and it uses hash-based routing
to have multiple pages. The advantage is that scouters can now change pages without fear of needing Wi-Fi access.

Backend

Likely the best part of this PR is that turbo-scout is no longer dependant on a single method of data transfer. The next section lists the methods that have been implemented so far, but several other methods are planned to be added before kickoff.

Supported data transfer ('share') methods

  • HTTP
  • Bluetooth
  • Local Filesystem
  • Apple Share
  • QR Code

michael-m-2983 and others added 9 commits September 1, 2024 20:59
* teleop & endgame questions

* Strategy & final comments

* gameplay (change performance to slider, change some things to paragraph or number)

---------

Co-authored-by: michael-m-2983 <116770284+michael-m-2983@users.noreply.github.com>
@michael-m-2983 michael-m-2983 added documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file labels Sep 21, 2024
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pylintpython3 (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pylint (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@michael-m-2983 michael-m-2983 merged commit 8afc53d into main Sep 27, 2024
@michael-m-2983 michael-m-2983 deleted the improved branch September 27, 2024 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants