Skip to content

Update show_QR #31

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

Merged
merged 3 commits into from
May 2, 2019
Merged

Update show_QR #31

merged 3 commits into from
May 2, 2019

Conversation

caternuson
Copy link
Contributor

Just an idea for #30

@ladyada
Copy link
Member

ladyada commented May 2, 2019

ill take anything cause current code isnt working at all :D

@caternuson
Copy link
Contributor Author

Sure. Was this suppose to be any fancier than just "clear screen and show QR code"?

@ladyada
Copy link
Member

ladyada commented May 2, 2019

its pretty simple, yeah

@caternuson
Copy link
Contributor Author

Generally works. This:

Adafruit CircuitPython 4.0.0-rc.1 on 2019-04-23; Adafruit PyPortal with samd51j20
>>> from adafruit_pyportal import PyPortal
>>> pp = PyPortal()
ESP firmware: bytearray(b'1.2.2\x00')
Set background to  0
No SD card found: no SD card
>>> pp.show_QR(b'hello world', 8, 20, 20)

gives this:
qr_pyportal
which was tested here:
https://zxing.org/w/decode.jspx
and spit back expected result:
qr_test

Note weird border quirk might be related to this:
adafruit/circuitpython#1839

@ladyada
Copy link
Member

ladyada commented May 2, 2019

fine by me!

@ladyada ladyada merged commit aeef800 into adafruit:master May 2, 2019
@caternuson
Copy link
Contributor Author

@ladyada want me to version bump this?

@ladyada
Copy link
Member

ladyada commented May 3, 2019

yes plz!

@caternuson
Copy link
Contributor Author

Done! Bumped major ver since the function call changed.

@ladyada
Copy link
Member

ladyada commented May 3, 2019

amazing, thank you thank you carter :)

adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request May 3, 2019
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.

2 participants