Skip to content

Conversation

@ahresse
Copy link
Contributor

@ahresse ahresse commented Jun 11, 2024

There was an issue when using the mqtt-all.py example with my enviro+. My display wasn't displaying the right info: There was a problem with the bounding box calculation of the message.

This PR is intended to fix this issue.

ahresse added 2 commits June 17, 2024 16:20
The previous display box function wasn't properly used.
This fix the information displayed on the enviro+ display.

Signed-off-by: Alexandre Esse <alexandre.esse.dev@gmail.com>
Signed-off-by: Alexandre Esse <alexandre.esse.dev@gmail.com>
@ahresse ahresse force-pushed the fix-mqtt-display branch from 9d81273 to f32e560 Compare June 17, 2024 14:20
@coveralls
Copy link

Pull Request Test Coverage Report for Build 21864144358

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 94.845%

Totals Coverage Status
Change from base Build 16221954147: 0.0%
Covered Lines: 276
Relevant Lines: 291

💛 - Coveralls

@helgibbons
Copy link
Contributor

Looks good - thanks for the PR, and sorry that it took us so long to merge it!

@helgibbons helgibbons merged commit 2bf11f2 into pimoroni:main Feb 10, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants