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

Add camera choice option to QR scanner field #12109

Merged
merged 22 commits into from
Nov 6, 2023

Conversation

zedascouves
Copy link
Contributor

Description

Allows selecting the camera for the QR scanner field. Issue #11459.

Addresses:

  • https://github.com/Budibase/budibase/issues/11459

Documentation

  • I have reviewed the budibase documentatation to verify if this feature requires any changes. If changes or new docs are required I have written them.

Added used camera preference option.
Added preferred camera setting.
Added preferred camera setting.
Copy link
Member

@aptkingston aptkingston left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks for the contribution @zedascouves.

@codecov-commenter
Copy link

codecov-commenter commented Oct 19, 2023

Codecov Report

Merging #12109 (fbbc5d0) into master (f21adde) will decrease coverage by 28.92%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

❗ Current head fbbc5d0 differs from pull request most recent head 511a377. Consider uploading reports for the commit 511a377 to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@             Coverage Diff             @@
##           master   #12109       +/-   ##
===========================================
- Coverage   75.74%   46.82%   -28.92%     
===========================================
  Files         338      164      -174     
  Lines       14186     5811     -8375     
  Branches     2981     1164     -1817     
===========================================
- Hits        10745     2721     -8024     
+ Misses       3218     2824      -394     
- Partials      223      266       +43     

see 502 files with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

Updated the code as suggested. Thanks!
Copy link
Contributor Author

@zedascouves zedascouves left a comment

Choose a reason for hiding this comment

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

I've changed the code as suggested.

@aptkingston aptkingston enabled auto-merge October 20, 2023 13:16
@aptkingston aptkingston disabled auto-merge October 20, 2023 13:16
@zedascouves
Copy link
Contributor Author

Do I need to do anything else for this to be successfully merged?

@shogunpurple shogunpurple disabled auto-merge November 6, 2023 18:02
@shogunpurple shogunpurple merged commit 6a76fb4 into Budibase:master Nov 6, 2023
1 of 2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants