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

Text encoding parameter #90

Merged
merged 3 commits into from
Sep 11, 2020
Merged

Text encoding parameter #90

merged 3 commits into from
Sep 11, 2020

Conversation

skjiisa
Copy link
Contributor

@skjiisa skjiisa commented Apr 30, 2020

Adds a contentEncoding parameter to generateQRCode allowing the user to specify which text encoding method to use when converting the content string into Data. The default encoding method is UTF-8, the same as the encoding method used before.

This only adds the parameter to the function and does not add an option in any of the example app interfaces.

Closes #89

@@ -692,9 +703,10 @@ public class EFQRCodeGenerator: NSObject {
return nil
}
let finalInputCorrectionLevel = inputCorrectionLevel
let finalContentEncoding = contentEncoding
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we really need this here?

@EyreFree EyreFree force-pushed the master branch 4 times, most recently from 8f2024e to b161195 Compare July 6, 2020 06:12
@EyreFree EyreFree merged commit 147cd1c into EFPrefix:master Sep 11, 2020
@EyreFree
Copy link
Member

@isvvc

Thanks for your contribution, 🙏

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.

ASCII or other encoding options
3 participants