-
Notifications
You must be signed in to change notification settings - Fork 79
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
QR code scanner (QML component) #9464
Conversation
@0x-r4bbit, could you please test this un Windows? |
Jenkins BuildsClick to see older builds (105)
|
@igor-sirotin unfortunately, I don't have a window machine |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome amount of work, some minor things inline
4865d33
to
a183c31
Compare
@caybro, it seems that I pushed my changes from other branch to this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks better :)
@caybro, thanks for comments, fixed that! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool!
It works! Although, it took a while to find the right angle 😅 qr-scanner-2023-02-07_09.16.14.mp4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The import check is broken because StatusQ knows nothing about DoOtherSide. This needs to be addressed, following #9125 rationale.
d0ac131
to
26eb226
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A big question: do we need this in DOtherSide? A better place would imho be status-desktop/ui/StatusQ/src
, now that we decoupled those
cc @micieslak
3e503e4
to
b12c16a
Compare
d10bcd2
to
d061fcc
Compare
Very cool PR! can confirm it works on linux! Edit:
Will try that feature :) (although i don't know where it is) |
Great job! It works for me from Sandbox, but unfortunately there are still problems to run the main app:
I was playing a bit to solve that but so far no results. Some nim experts are needed here, I suppose... |
If we want to use it in the main app (and we do, I suppose), we need to compile it via DOtherSide for the main app. At least I can't see any better solution for now. |
Definitely needs a rebase |
99b6515
to
b1bb4ac
Compare
This should build in Linux now! |
Nope...
I don't know what on earth needs a Qt ODBC driver but it's not found |
Interestingly enough, if I execute
and no mention of libodbc.so |
But this is not the issue; you want to look at the runtime dependencies of the
|
In other words, this seems more like a packaging problem, where we ship all Qt plugins but some of them (namely the libqsqlodbc.so plugin which we don't even need in our app) have unsatisfied runtime deps |
Interesting! i tried to build the app without using the lib and i'm running into the same error! |
aceefba
to
9372a3c
Compare
Required for #5837
Overrides:
What does the PR do
This is a QR code scanner. Based on QZXing.
Notes:
Didn't fix it yet, will try to improve it later.
Will try to fix it later.
Currently pointing to status-im fork.
Affected areas
StatusQ
StatusQ checklist
Screenshot of functionality (including design for comparison)
2023-02-06.18.25.23.mov