-
Notifications
You must be signed in to change notification settings - Fork 53
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: use react native linking instead of branch #4038
Merged
Merged
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
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
johnsmith-gooddollar
approved these changes
Jul 24, 2023
cryptoKevinL
pushed a commit
to Wallet-Chat/GoodDAPP
that referenced
this pull request
Aug 16, 2023
* add: use react native linking instead of branch * Update deepLinking.native.js * fix: selective deeplinks --------- Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> Co-authored-by: LewisB <lewis@gooddollar.org>
sirpy
added a commit
that referenced
this pull request
Sep 3, 2023
* added WC * new commit * Updated for providing proper wallet info to walletchat widget * updating walletchat widget version * attempts to sign message - have to do it a few times before UI loads correectly * chat with owner * added chat icon * correctly prints fromAddr when a TX is clicked on in console, but is not displaying it correctly yet * Revert "correctly prints fromAddr when a TX is clicked on in console, but is not displaying it correctly yet" This reverts commit ac08fb7. * found fromAddr in existing data * better for printing shortened address * adding toAddress and fromAddress to the event data in the feed * working on chat with owner * fixed clickable issue for chat with owner and added owneraddress * hotfix: correct browser check logic * hotfix: feed_update value and prevent multi calls * Update branch, add latest language files (#3871) Co-authored-by: LewisB <lewis@gooddollar.org> * make version different on dev/qa/prod * chore: release qa version 2.14.1-0 [skip build] * chore: release prod version 2.15.0 [skip build] * disable QA release to google play * chore: release qa version 2.15.1-0 [skip build] * chore: release prod version 2.16.0 [skip build] * disable intent filter * chore: bump version * Update .env.prod * chore: release qa version 2.19.1-0 [skip build] * chore: release prod version 2.20.0 [skip build] * wip: vercel rewrite only test * hotfix: fix posthot proxy * hotfix: temporary fix for posthog vercel redirects. remove trailing slash * add: use react native linking instead of branch (#4038) * add: use react native linking instead of branch * Update deepLinking.native.js * fix: selective deeplinks --------- Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> Co-authored-by: LewisB <lewis@gooddollar.org> * hotfix: use random rpc * (Bug) No redirect from "I Give Up" screen on Safari desktop + mobile (#4047) * hotfix: temporary fix for posthog vercel redirects. remove trailing slash * wip: giveup redirect fallback for requestIdleCallback * Update GiveUpButton.js * Update GiveUpButton.js * formatting --------- Co-authored-by: sirpy <hadarr@gmail.com> Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> * hotfix: correct rpc error handling and randomize * (Bug): add optional eip namespaces to wc approve response (#4048) * add optional eip namespaces to wc approve response * add listeners for session_expire/session_update * remove duplicate session_update definition * Apply suggestions from code review * Update src/lib/wallet/WalletConnectClient.js --------- Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> * add: remove no longer required patch for vercel * 4037 add webview check fv (#4043) * fix: mustache break loading explanationDialog * wip: add unsupported browser modal to fv flow * hotfix: temporary fix for posthog vercel redirects. remove trailing slash * fix isWebView check, add copy url for regular flow * add webview check to fv-standalone * fix: don't apply isWebView on all useCameraSupport calls * use DeepLinking instead of window for copy url * chore: fix snapshot * Update src/components/browserSupport/hooks/useCameraSupport.js * replace useEffect with camera support hook --------- Co-authored-by: sirpy <hadarr@gmail.com> Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> * (Bug): wallet-connect trying to reconnect when signing tx (#4049) * wip: web fix for wc trying to reconnect * apply alternative solution for reconnect on incoming wc link --------- Co-authored-by: sirpy <hadar@gooddollar.org> * (fix) - android action-flow: check against env name instead of github.ref_name * chore: release qa version 2.20.1-0 [skip build] * hotfix: correctly handle rpc null result * (fix) - fv-standalone: unsupported modal not passed down correctly * fix: use env_name also for vercel pull * fix: replace branch names with env names * fix typo * final fix android flow * fix android flow * Fix: merge previous namespaces list with new namespaces (#4056) * merge namespaces before with new namespaces * only update session with eip155 chain namespaces * remove unused var * fix: only filter on updated eip155, run update instead of updateSession * only log info on session_update for now * chore: release qa version 2.20.1-1 [skip build] * fix: use sessionTopic on native to decide if new connection should be made (#4058) * fix: use sessionTopic on native to decide on connect or not * fix if check * add just verifying if incoming link is a sign request * better variable description * temp: disable distribute to stores when on prod * remove unused var * fix: make sure wcIncomingLink is set * chore: release qa version 2.20.1-2 [skip build] * chore: release prod version 2.21.0 [skip build] * fix: android prod env check * restore prod release * add code push debug flag, show appcenter client version, remove quotes from token value * chore: release qa version 2.21.1-0 [skip build] * chore: release prod version 2.22.0 [skip build] * remove code push debug flags * fix: #4031 - wc request modal does not fit mobile screen * fix: from decimals when input not in right decimals * hotfix: eth_sign personal_sign * resized widget height * updating to auto-log in * updated widget * added chat with owner button to tx reciept * testing some additional prints in react-native widget * updating widget for wake-up mechanism * cleanup hardcoded stuff * remove image * updating widget version to add a print * changed whatsapp chat icon to normal chat icon and update WC package * change fromAddress back to Uknown * changed fromAddy to Unknown and changed the src for the icon * created a hook that returns the web3Provider from the goodwalletprovider * made requested changes * remove console log --------- Co-authored-by: Cyber Kevin <emailkevinlarson@gmail.com> Co-authored-by: sirpy <hadarr@gmail.com> Co-authored-by: sirpy <hadar@gooddollar.org> Co-authored-by: LewisB <lewis@gooddollar.org> Co-authored-by: John Smith <alexey@gooddollar.org> Co-authored-by: Techadmin <techadmin@gooddollar.org> Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> Co-authored-by: Lewis B <laurence@gooddollar.org>
L03TJ3
added a commit
that referenced
this pull request
Oct 17, 2023
* added WC * new commit * Updated for providing proper wallet info to walletchat widget * updating walletchat widget version * attempts to sign message - have to do it a few times before UI loads correectly * chat with owner * added chat icon * correctly prints fromAddr when a TX is clicked on in console, but is not displaying it correctly yet * Revert "correctly prints fromAddr when a TX is clicked on in console, but is not displaying it correctly yet" This reverts commit ac08fb7. * found fromAddr in existing data * better for printing shortened address * adding toAddress and fromAddress to the event data in the feed * working on chat with owner * fixed clickable issue for chat with owner and added owneraddress * hotfix: correct browser check logic * hotfix: feed_update value and prevent multi calls * Update branch, add latest language files (#3871) Co-authored-by: LewisB <lewis@gooddollar.org> * make version different on dev/qa/prod * chore: release qa version 2.14.1-0 [skip build] * chore: release prod version 2.15.0 [skip build] * disable QA release to google play * chore: release qa version 2.15.1-0 [skip build] * chore: release prod version 2.16.0 [skip build] * disable intent filter * chore: bump version * Update .env.prod * chore: release qa version 2.19.1-0 [skip build] * chore: release prod version 2.20.0 [skip build] * wip: vercel rewrite only test * hotfix: fix posthot proxy * hotfix: temporary fix for posthog vercel redirects. remove trailing slash * add: use react native linking instead of branch (#4038) * add: use react native linking instead of branch * Update deepLinking.native.js * fix: selective deeplinks --------- Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> Co-authored-by: LewisB <lewis@gooddollar.org> * hotfix: use random rpc * (Bug) No redirect from "I Give Up" screen on Safari desktop + mobile (#4047) * hotfix: temporary fix for posthog vercel redirects. remove trailing slash * wip: giveup redirect fallback for requestIdleCallback * Update GiveUpButton.js * Update GiveUpButton.js * formatting --------- Co-authored-by: sirpy <hadarr@gmail.com> Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> * hotfix: correct rpc error handling and randomize * (Bug): add optional eip namespaces to wc approve response (#4048) * add optional eip namespaces to wc approve response * add listeners for session_expire/session_update * remove duplicate session_update definition * Apply suggestions from code review * Update src/lib/wallet/WalletConnectClient.js --------- Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> * add: remove no longer required patch for vercel * 4037 add webview check fv (#4043) * fix: mustache break loading explanationDialog * wip: add unsupported browser modal to fv flow * hotfix: temporary fix for posthog vercel redirects. remove trailing slash * fix isWebView check, add copy url for regular flow * add webview check to fv-standalone * fix: don't apply isWebView on all useCameraSupport calls * use DeepLinking instead of window for copy url * chore: fix snapshot * Update src/components/browserSupport/hooks/useCameraSupport.js * replace useEffect with camera support hook --------- Co-authored-by: sirpy <hadarr@gmail.com> Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> * (Bug): wallet-connect trying to reconnect when signing tx (#4049) * wip: web fix for wc trying to reconnect * apply alternative solution for reconnect on incoming wc link --------- Co-authored-by: sirpy <hadar@gooddollar.org> * (fix) - android action-flow: check against env name instead of github.ref_name * chore: release qa version 2.20.1-0 [skip build] * hotfix: correctly handle rpc null result * (fix) - fv-standalone: unsupported modal not passed down correctly * fix: use env_name also for vercel pull * fix: replace branch names with env names * fix typo * final fix android flow * fix android flow * Fix: merge previous namespaces list with new namespaces (#4056) * merge namespaces before with new namespaces * only update session with eip155 chain namespaces * remove unused var * fix: only filter on updated eip155, run update instead of updateSession * only log info on session_update for now * chore: release qa version 2.20.1-1 [skip build] * fix: use sessionTopic on native to decide if new connection should be made (#4058) * fix: use sessionTopic on native to decide on connect or not * fix if check * add just verifying if incoming link is a sign request * better variable description * temp: disable distribute to stores when on prod * remove unused var * fix: make sure wcIncomingLink is set * chore: release qa version 2.20.1-2 [skip build] * chore: release prod version 2.21.0 [skip build] * fix: android prod env check * restore prod release * add code push debug flag, show appcenter client version, remove quotes from token value * chore: release qa version 2.21.1-0 [skip build] * chore: release prod version 2.22.0 [skip build] * remove code push debug flags * fix: #4031 - wc request modal does not fit mobile screen * fix: from decimals when input not in right decimals * hotfix: eth_sign personal_sign * resized widget height * updating to auto-log in * updated widget * added chat with owner button to tx reciept * testing some additional prints in react-native widget * updating widget for wake-up mechanism * cleanup hardcoded stuff * remove image * updating widget version to add a print * changed whatsapp chat icon to normal chat icon and update WC package * change fromAddress back to Uknown * changed fromAddy to Unknown and changed the src for the icon * created a hook that returns the web3Provider from the goodwalletprovider * made requested changes * remove console log * fixed bottom navigation issue and WC icon * fixed some styling issues in dashboard * fix styling issue * adjust WC icon * updates to use newer react-native WC widget for android native changes * made requested changes * fix: make a helper for txtype regex --------- Co-authored-by: Cyber Kevin <emailkevinlarson@gmail.com> Co-authored-by: sirpy <hadarr@gmail.com> Co-authored-by: sirpy <hadar@gooddollar.org> Co-authored-by: LewisB <lewis@gooddollar.org> Co-authored-by: John Smith <alexey@gooddollar.org> Co-authored-by: Techadmin <techadmin@gooddollar.org> Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> Co-authored-by: Lewis B <laurence@gooddollar.org> Co-authored-by: Lewis B <lewis@ikigaistudios.eu>
L03TJ3
added a commit
that referenced
this pull request
Nov 17, 2023
* added WC * new commit * Updated for providing proper wallet info to walletchat widget * updating walletchat widget version * attempts to sign message - have to do it a few times before UI loads correectly * chat with owner * added chat icon * correctly prints fromAddr when a TX is clicked on in console, but is not displaying it correctly yet * Revert "correctly prints fromAddr when a TX is clicked on in console, but is not displaying it correctly yet" This reverts commit ac08fb7. * found fromAddr in existing data * better for printing shortened address * adding toAddress and fromAddress to the event data in the feed * working on chat with owner * fixed clickable issue for chat with owner and added owneraddress * hotfix: correct browser check logic * hotfix: feed_update value and prevent multi calls * Update branch, add latest language files (#3871) Co-authored-by: LewisB <lewis@gooddollar.org> * make version different on dev/qa/prod * chore: release qa version 2.14.1-0 [skip build] * chore: release prod version 2.15.0 [skip build] * disable QA release to google play * chore: release qa version 2.15.1-0 [skip build] * chore: release prod version 2.16.0 [skip build] * disable intent filter * chore: bump version * Update .env.prod * chore: release qa version 2.19.1-0 [skip build] * chore: release prod version 2.20.0 [skip build] * wip: vercel rewrite only test * hotfix: fix posthot proxy * hotfix: temporary fix for posthog vercel redirects. remove trailing slash * add: use react native linking instead of branch (#4038) * add: use react native linking instead of branch * Update deepLinking.native.js * fix: selective deeplinks --------- Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> Co-authored-by: LewisB <lewis@gooddollar.org> * hotfix: use random rpc * (Bug) No redirect from "I Give Up" screen on Safari desktop + mobile (#4047) * hotfix: temporary fix for posthog vercel redirects. remove trailing slash * wip: giveup redirect fallback for requestIdleCallback * Update GiveUpButton.js * Update GiveUpButton.js * formatting --------- Co-authored-by: sirpy <hadarr@gmail.com> Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> * hotfix: correct rpc error handling and randomize * (Bug): add optional eip namespaces to wc approve response (#4048) * add optional eip namespaces to wc approve response * add listeners for session_expire/session_update * remove duplicate session_update definition * Apply suggestions from code review * Update src/lib/wallet/WalletConnectClient.js --------- Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> * add: remove no longer required patch for vercel * 4037 add webview check fv (#4043) * fix: mustache break loading explanationDialog * wip: add unsupported browser modal to fv flow * hotfix: temporary fix for posthog vercel redirects. remove trailing slash * fix isWebView check, add copy url for regular flow * add webview check to fv-standalone * fix: don't apply isWebView on all useCameraSupport calls * use DeepLinking instead of window for copy url * chore: fix snapshot * Update src/components/browserSupport/hooks/useCameraSupport.js * replace useEffect with camera support hook --------- Co-authored-by: sirpy <hadarr@gmail.com> Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> * (Bug): wallet-connect trying to reconnect when signing tx (#4049) * wip: web fix for wc trying to reconnect * apply alternative solution for reconnect on incoming wc link --------- Co-authored-by: sirpy <hadar@gooddollar.org> * (fix) - android action-flow: check against env name instead of github.ref_name * chore: release qa version 2.20.1-0 [skip build] * hotfix: correctly handle rpc null result * (fix) - fv-standalone: unsupported modal not passed down correctly * fix: use env_name also for vercel pull * fix: replace branch names with env names * fix typo * final fix android flow * fix android flow * Fix: merge previous namespaces list with new namespaces (#4056) * merge namespaces before with new namespaces * only update session with eip155 chain namespaces * remove unused var * fix: only filter on updated eip155, run update instead of updateSession * only log info on session_update for now * chore: release qa version 2.20.1-1 [skip build] * fix: use sessionTopic on native to decide if new connection should be made (#4058) * fix: use sessionTopic on native to decide on connect or not * fix if check * add just verifying if incoming link is a sign request * better variable description * temp: disable distribute to stores when on prod * remove unused var * fix: make sure wcIncomingLink is set * chore: release qa version 2.20.1-2 [skip build] * chore: release prod version 2.21.0 [skip build] * fix: android prod env check * restore prod release * add code push debug flag, show appcenter client version, remove quotes from token value * chore: release qa version 2.21.1-0 [skip build] * chore: release prod version 2.22.0 [skip build] * remove code push debug flags * fix: #4031 - wc request modal does not fit mobile screen * fix: from decimals when input not in right decimals * hotfix: eth_sign personal_sign * resized widget height * updating to auto-log in * updated widget * added chat with owner button to tx reciept * testing some additional prints in react-native widget * updating widget for wake-up mechanism * cleanup hardcoded stuff * remove image * updating widget version to add a print * changed whatsapp chat icon to normal chat icon and update WC package * change fromAddress back to Uknown * changed fromAddy to Unknown and changed the src for the icon * created a hook that returns the web3Provider from the goodwalletprovider * made requested changes * remove console log * fixed bottom navigation issue and WC icon * fixed some styling issues in dashboard * fix styling issue * adjust WC icon * updates to use newer react-native WC widget for android native changes * made requested changes * fix: make a helper for txtype regex * new commit * fix import * added lewis recommended code and updated our package * fix: styling * update to latest NPM package for widget * fix: snapshot --------- Co-authored-by: Cyber Kevin <emailkevinlarson@gmail.com> Co-authored-by: sirpy <hadarr@gmail.com> Co-authored-by: sirpy <hadar@gooddollar.org> Co-authored-by: LewisB <lewis@gooddollar.org> Co-authored-by: John Smith <alexey@gooddollar.org> Co-authored-by: Techadmin <techadmin@gooddollar.org> Co-authored-by: johnsmith-gooddollar <89783679+johnsmith-gooddollar@users.noreply.github.com> Co-authored-by: Lewis B <laurence@gooddollar.org> Co-authored-by: Lewis B <lewis@ikigaistudios.eu>
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.
Description
Branch.io is starting to charge money for the basic usage.
Since we don't actually use branch links we dont really need it.
Replaced with basic react universal links handling.