CHANGE: upgrade camera package to react-native-vision-camera FIX: [APP-171], [APP-776] #5178
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
New, updated, and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: react-native-camera@4.2.1 |
mikedemarais
reviewed
Nov 7, 2023
…brody/QR-code-scanner-fixes * 'develop' of https://github.com/rainbow-me/rainbow: [PATCH]: react-native-fast-image iOS 17 spec (#5188) Feature: Add remote promo sheet (#5140) mints: fix insufficient eth check (#5186) sends: fix prev sends logic (#5185) Update copy for an Error Occurred message for Support (#5187) Fixes for activity list coin icons and migrating some components to network instead of assetType (#5143) audit: axios + reservior client bump (#5184) bump to v1.9.10 (#5183) nonces: reset nonces (#5170) Cleanup unused coinrow components (#5142) reviews: bump watch wallet threshold (#5181) lang: fix english revert (#5182) WC: enabled eth_sign but only for connections (#5180) wc: use methodRegistry on all networks (#5179) bump to v1.9.9 (#5173) walletconnect: sanitize 712 signs (#5159)
APP-171 camera light is staying on after making a WC link in Rainbow
The camera light is staying on after making a WC link in Rainbow Anyone else see that? is keeping the camera on a bug? Steps to reproduce - connect to a site like RainbowKit.com using WalletConnect and the Rainbow app. |
BrodyHughes
requested review from
benisgold,
skylarbarrera and
jinchung
as code owners
BrodyHughes
changed the title
…brody/QR-code-scanner-fixes * 'develop' of https://github.com/rainbow-me/rainbow: v bump 1.9.11 (#5204) Simulation fixes (#5194) [Android]: Random bug fixes (#5195) revert onpress (#5193) wc: tx simulation (#5177) [APP-917]: Minimum recieved shown in wei not in ETH on eth to weth swap on Arbitrum in latest iOS prod (#5192) RPC Proxy (#5169) . (#5191) [Android] Fix: Hide Navigation & Status bar (#5150)
…brody/QR-code-scanner-fixes
…brody/QR-code-scanner-fixes
BrodyHughes
force-pushed
the
brody/QR-code-scanner-fixes
branch
from
c20af49
to
02c503b
Compare
BrodyHughes
force-pushed
the
brody/QR-code-scanner-fixes
branch
from
02c503b
to
f816cbb
Compare
BrodyHughes
force-pushed
the
brody/QR-code-scanner-fixes
branch
from
81d6c26
to
94b57f5
Compare
dereknelson
approved these changes
Jan 17, 2024
skylarbarrera
approved these changes
Jan 17, 2024
|
Kinda Sus intermediate screen on android. Not sure if this is expected but doesn't seem like it. |
…brody/QR-code-scanner-fixes * 'develop' of https://github.com/rainbow-me/rainbow: bump react-native-linear-gradient (#5328)
walmat
added a commit
that referenced
this pull request
Jan 26, 2024
* [APP-1051]: (feat): add backups global config (#5306) * (feat): add backups global config * stash * add migration for setting initial backup method * add method to migrations array * revert pbx change * change backup status value * chore: hide testnet settings (#5308) * remote testnet settings * trying this * switch to breadcrumbs (#5309) * switch to breadcrumbs * change to debug calls * . (#5311) * bump version to v1.9.14 (#5312) * [APP-1052]: Add icon to Settings depending on `preferredBackupMethod` and `backupStatus` (#5313) * [APP-1052]: (feat): adjust settings page row * fix migration and hook for useWallets * simplify an if statement * chore: fix CI (#5307) * disable deeplinks jest tests * try this * disable * oop * anotha 1 * oop * swap changes * e2e for swap flow changes * testing * yes * revert file names * yup * audit: follow-redirects (#5323) * audit mmkv (#5325) * wallets and backups screen progress (#5315) * wallets and backups screen progress * manual backups progress * progress * feat(backups): more work on wallets and backups * revert pbx * rev init wallet change * final touches on wallets and backup view * f (#5327) * bump react-native-linear-gradient (#5328) * CHANGE: upgrade camera package to react-native-vision-camera FIX: [APP-171], [APP-776] (#5178) * init * init * more * GRR!!! BARK BARK * fixes * revert * fix * changes * changes * hoist * hook * comment * fix * i actually did it * cleanup 1 * get android build working * remove old package * move back to hook * . * . * . * build * build fix * provisioning profiles * bump ios version * . * . * oops * language change * final fixes * Update project.pbxproj --------- Co-authored-by: Ben Goldberg <bengoldberg@rainbow.me> Co-authored-by: skylarbarrera <skylar.barrera@gmail.com> * [APP-925]: Backend Driven Cards (#5198) * add experimental flag for remote cards (#5196) * [APP-929]: Add `RemoteCardProvider` and react-query calls (#5197) * add card and cardCollection to arc * rm logs * [APP-930]: Add `<RemoteCard />` component (#5203) * start progress on remote card * update remote card * [APP-932]: Add `<RemoteCardCarousel />` component (#5205) * added card carousel to wallet and discover screen * update card and package * tweaks to make carousel fit in * update styles * idkwtfiad * disable scroll when only one card is present and adjust view dimensions size based on card length * . * fix layout issues * remove unused debuglayout * fix card and re-enable check to not show cards again * Apply suggestions from code review * tweak discover page * add some testing edge cases * add remote config stuff * fix graphql schema * code review changes * change to new remote config hook * fix remote config * new design * start with view weekly earnings flow * progress on points breakdown from last week * Delete ios/.xcode.env.local * Update src/components/asset-list/RecyclerAssetList2/core/ViewDimensions.tsx * Update src/components/asset-list/RecyclerAssetList2/core/getLayoutProvider.tsx * Apply suggestions from code review * chore: rm snap carousel * Update shim.js * finish earnings screen * finish spec * adjust gutter size depending on how many cards are in the carousel * clean unused import * fix font weight on android and conditionally wrap remote card if primary button has action * fix recycler list view blank space for watched wallets * adjust card to prev card on dismissal * fix text formatting * fix: lint * change gutter size * fix text truncation * fix ios touch event overlap * fix: disable hiding cards in dev * Update src/components/cards/remote-cards/RemoteCardProvider.tsx * weekly earnings fix and leaderboard diff (#5330) * fix weekly earnings total * leaderboard ranking diff * Update src/screens/points/content/PointsContent.tsx Co-authored-by: Matthew Wall <matthew.wallt@gmail.com> --------- Co-authored-by: Matthew Wall <matthew.wallt@gmail.com> * filter & reduce --------- Co-authored-by: Ben Goldberg <bengoldberg@rainbow.me> Co-authored-by: skylarbarrera <skylar.barrera@gmail.com> * bump: fastlane (#5331) * Points: rank card fixes (#5332) * fixes * fix ens display * Points: notifications toggle (#5329) * cleanup unused stuff and progress on backups flow * rename points topic (#5334) * rename points topic * Update constants.ts --------- Co-authored-by: Skylar Barrera <skylar.barrera@gmail.com> * backups: warning + new secret phrase styles (#5322) * save progress * cleanup and finish secret phrase / private key viewiing screens --------- Co-authored-by: Matthew Wall <matthew.wallt@gmail.com> * chore: cleanup WalletsAndBackup settings screen and replace lang with i18n * bump minSdkVersion (#5337) * android settings fixes * progress on cloud backup flow * cloud backup flow progress --------- Co-authored-by: Skylar Barrera <skylar.barrera@gmail.com> Co-authored-by: brdy <41711440+BrodyHughes@users.noreply.github.com> Co-authored-by: Ibrahim Taveras <ibrahim@rainbow.me> Co-authored-by: Ben Goldberg <bengoldberg@rainbow.me>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes APP-171
Goals:
react-native-vision-cameraI believe this PR accomplishes all three of these. Things I also had to do to make the new package work:
Remaining concerns for eng:
POW:
In the video I test closing by swiping and by the 'x' button. The green status light and the torch both properly turn off.
RPReplay_Final1704226814.MP4