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

fix: issue reported after location mqtt implementation #278

Merged
merged 2 commits into from
Mar 29, 2024

Conversation

tobiasehlert
Copy link
Owner

@tobiasehlert tobiasehlert commented Mar 29, 2024

Since we already have latitude and longitude data in our CarGeodata struct, I don't see us using the combined value in the end. So we'll stick with only adding an case-entry for location and we don't get the current errors anymore.

fix #277

@tobiasehlert tobiasehlert added bug Something isn't working enhancement New feature or request go Pull requests that update Go code WIP Work in progress labels Mar 29, 2024
@tobiasehlert tobiasehlert self-assigned this Mar 29, 2024
Copy link

sonarcloud bot commented Mar 29, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@tobiasehlert tobiasehlert marked this pull request as ready for review March 29, 2024 23:03
@tobiasehlert tobiasehlert changed the title feat: add location from mqtt fix: issue reported after location mqtt implementation Mar 29, 2024
@tobiasehlert tobiasehlert merged commit 4636fe6 into main Mar 29, 2024
4 checks passed
@tobiasehlert tobiasehlert deleted the feature-mqtt-location-topic branch March 29, 2024 23:11
@tobiasehlert tobiasehlert removed bug Something isn't working enhancement New feature or request WIP Work in progress labels Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Pull requests that update Go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add mqtt location topic support
1 participant