Adding BLE support on Android Device + DPAD controller Connection#330
Merged
Conversation
… to ble device and managing devices.
Collaborator
|
|
Collaborator
Author
added null checks to fix the failures |
Bumps [express](https://github.com/expressjs/express) from 4.17.1 to 4.18.2. - [Release notes](https://github.com/expressjs/express/releases) - [Changelog](https://github.com/expressjs/express/blob/master/History.md) - [Commits](expressjs/express@4.17.1...4.18.2) --- updated-dependencies: - dependency-name: express dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [ua-parser-js](https://github.com/faisalman/ua-parser-js) from 0.7.23 to 0.7.33. - [Release notes](https://github.com/faisalman/ua-parser-js/releases) - [Changelog](https://github.com/faisalman/ua-parser-js/blob/master/changelog.md) - [Commits](faisalman/ua-parser-js@0.7.23...0.7.33) --- updated-dependencies: - dependency-name: ua-parser-js dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
… to ble device and managing devices.
Collaborator
|
Added a few more fixes. Also noticed that the jpeg images for the README are not pushed to this PR. Please add them after compressing them. |
…into android-ble
…et back from bluetooth device list screen
thias15
reviewed
Apr 3, 2023
Comment on lines
278
to
279
Collaborator
There was a problem hiding this comment.
Why are these set to null here?
Collaborator
Author
There was a problem hiding this comment.
So, there was a bug into the application, which is being fixed here by reinitializing the variables.
Bug was if you turn on the auto mode into the autopilot, then move to settings by clicking on the usb/ble icon and come back to the fragment.
it crashes.
…nt usb icon and remove usb connection activities from settings fragment
thias15
reviewed
Apr 5, 2023
thias15
reviewed
Apr 6, 2023
thias15
left a comment
thias15
left a comment
Collaborator
There was a problem hiding this comment.
Looks good, tested. Just need to adjust README with updated text and screenshots.
lacykaltgr
pushed a commit
to satinavrobotics/SatiBot
that referenced
this pull request
Jun 1, 2025
…-f#330) * adding all the changes for bluetooth for ble connection, sending data to ble device and managing devices. * commenting extra toasts * adding null checks * upgrade android api * upgrade android sdk version of controller and gradle lint build failed issue solved * update python style * Bump express from 4.17.1 to 4.18.2 in /policy/frontend Bumps [express](https://github.com/expressjs/express) from 4.17.1 to 4.18.2. - [Release notes](https://github.com/expressjs/express/releases) - [Changelog](https://github.com/expressjs/express/blob/master/History.md) - [Commits](expressjs/express@4.17.1...4.18.2) --- updated-dependencies: - dependency-name: express dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * Bump ua-parser-js from 0.7.23 to 0.7.33 in /policy/frontend Bumps [ua-parser-js](https://github.com/faisalman/ua-parser-js) from 0.7.23 to 0.7.33. - [Release notes](https://github.com/faisalman/ua-parser-js/releases) - [Changelog](https://github.com/faisalman/ua-parser-js/blob/master/changelog.md) - [Commits](faisalman/ua-parser-js@0.7.23...0.7.33) --- updated-dependencies: - dependency-name: ua-parser-js dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * adding all the changes for bluetooth for ble connection, sending data to ble device and managing devices. * cleanup * update firmware * ui fixes * fix bluetooth connecting issue * fix in bluetooth connectivity in android 12 and above * fix issue in autopilot fragment and object navigation fragment when get back from bluetooth device list screen * fix readme images * push screen shots of ble connections * compress images and simplify README * finetuning * bt icon blue when connected * implement separate usb fragment and change navigation from all fragment usb icon and remove usb connection activities from settings fragment * apply app style * clean-up * use MAX_MSG_SZ * clean-up * increase MTU size in BLE and set BLE icon issue in Robot info screen * increase MTU size to get maximum string * minor tweaks * update readme * fix style --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Sanjeev Sharma <> Co-authored-by: thias15 <thias15@web.de> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.