-
Notifications
You must be signed in to change notification settings - Fork 984
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
Improve QR Code Component in Quo Preview Sandbox #15755
Conversation
7cae42c
to
7282c2c
Compare
Jenkins BuildsClick to see older builds (8)
|
7282c2c
to
93c34c7
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.
We've been trying to add component specs to quo2 components @siddarthkay. Would you mind writing some?
Apart from the missing test, I think the component looks good in my tests. Good stuff Sid!
(let [state (reagent/atom {:text "https://status.im" | ||
:error-correction-level :highest}) | ||
text (reagent/cursor state [:text]) | ||
error-correction-level (reagent/cursor state [:error-correction-level]) |
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.
I like the usage of cursors in these preview screens, I'll use your idea here in the future.
ddbe0e0
to
2685561
Compare
This PR adds the ability to : - Generate a QR by passing any url. - See different error correction levels. - Shows the media server url for that specific QR code. - Moved media server URI generation logic to `image_server.cljs`
2685561
to
c1a6e9f
Compare
This PR adds the ability to :
image_server.cljs
This is how it looks like:
Screen.Recording.2023-04-26.at.3.10.18.PM.mov
status: ready