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

Add option to remove some non-JSON MQTT topics #377

Merged
merged 2 commits into from
May 30, 2024

Conversation

iranl
Copy link
Collaborator

@iranl iranl commented May 28, 2024

No description provided.

@iranl iranl added the enhancement New feature or request label May 28, 2024
@iranl iranl added this to the 8.35 milestone May 28, 2024
@technyon
Copy link
Owner

Hi,

Why is the NimBLE log level changed. I guess this will create a larger binary due to more debug statements being compiled in.

@iranl
Copy link
Collaborator Author

iranl commented May 29, 2024

I lowered the level in the debug binaries because it creates to many messages about all the (non Nuki) BLE advertisements it sees making debugging in general only harder.

Release and debug are now both at 0, meaning no logging of NimBLE messages.

@technyon
Copy link
Owner

Ah yes sorry I read the wrong way. It's 4 to 0, not the other way round.

@technyon technyon merged commit ff741f7 into technyon:master May 30, 2024
2 checks passed
@iranl iranl deleted the add-option-to-remove-non-json branch May 30, 2024 17:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants