-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Sample code fails - "QRCode" doesn't exist #475
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
Comments
This repo isn't really being maintained- most have success just using the prior release of v1.4.1. It appears to be more stable. |
QRCode appears to have been removed. I changed my code to:
|
@melmullen @markdall This is actually better, as the old version provides a System.drawing.Bitmap. Which is already somewhat outdated and also limits us to the Windows platform. The new version gives us a byte array which we can use directly in skia etc. |
@kenan-kajkus Sure, and the documentation ought to be updated to reflect how to use the library, instead of the old way that doesn't work.... right? |
@markdall Sorry, I wasn't trying to excuse the lack of documentation. I just stumbled across this issue and wanted to explain why this change is an improvement despite breaking changes. |
Type of issue
[x ] Bug
Expected Behavior
Sample code from readme.md works in VS 2022 LTSC 17.6 / .NET 6.0
Sample code from readme.md:
Current Behavior
Possible Solution (optional)
Steps to Reproduce (for bugs)
Use the current sample code first found by starting at the top of the Readme.md file and scrolling down, in VS 2022. "QRCode" doesn't seem to exist anymore.
Your Environment
I'd make a pull request with my version that works (see: possible solution) but I'm not an actual developer and my solution may have everyone who reads it slapping their foreheads in dismay at my folly. I mostly just wanted to point out that the sample code doesn't work, likely because the name space or class name was changed.
The text was updated successfully, but these errors were encountered: