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.

Sign up for GitHub

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

Jump to bottom

[db_manager] fix unicode in SQL string #4888

Closed
wants to merge 2 commits into from
Closed

[db_manager] fix unicode in SQL string #4888

wants to merge 2 commits into from

Conversation

strk
Copy link
Contributor

@strk strk commented Jul 19, 2017

See https://issues.qgis.org/issues/16833
Includes testcase

@strk strk mentioned this pull request Jul 19, 2017
Closed
@strk strk requested a review from NathanW2 July 19, 2017 18:58
@strk strk self-assigned this Jul 19, 2017
@strk strk added the Bugfix label Jul 19, 2017
@strk strk added this to the 2.18 milestone Jul 19, 2017
@strk
Copy link
Contributor Author

strk commented Jul 19, 2017

Merged as of 3120068

@strk strk closed this Jul 19, 2017
@borysiasty
Copy link
Member

Thanks a lot @strk!

@MarHoff
Copy link

MarHoff commented Jul 20, 2017

Thanks for that quick fix!

@qgib qgib mentioned this pull request May 25, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NathanW2 NathanW2 Awaiting requested review from NathanW2

Assignees

@strk strk

Labels
None yet
Projects
None yet
Milestone
2.18.21
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@strk @borysiasty @MarHoff