-
Notifications
You must be signed in to change notification settings - Fork 8
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
feat: add btc map #262
feat: add btc map #262
Conversation
Overall it seems to work ok, map loads a bit slowly but could just be me. Links cannot be opened, is it just Expo go with this issue?
|
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.
tACK, it's not perfect but the code is isolated and possibly we can improve it over time
I think it's broke on BTCMap's end, I faced the same issue on Blink too (they also use BTCMap) |
pages/settings/BitcoinMap.tsx
Outdated
) : ( | ||
<WebView | ||
source={{ uri: mapUrl }} | ||
className="flex-1 bg-blue-100" |
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.
className="flex-1 bg-blue-100" | |
className="flex-1" |
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.
Added some comments, other than that:
tACK
Fixes #256
Modal is yet to be added. If location is not allowed it will just show a random location and then can be dragged
Screenshots
Tested on both android and iOS