Merge pull request #536 from bluesky-social/ansh/app-558-share-profile-and-post-broken-on-android

[APP-558] Sharing refactor
zio/stable
Ansh 2023-04-25 14:04:56 -07:00 committed by GitHub
commit c8a7f27d43
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 34 additions and 25 deletions

View File

@ -36,6 +36,13 @@
- Note that since 0.70, debugging using the old debugger (which shows up using CMD+D) doesn't work anymore. Follow the instructions below to debug the code: https://reactnative.dev/docs/next/hermes#debugging-js-on-hermes-using-google-chromes-devtools
### Developer Menu
To open the [Developer Menu](https://docs.expo.dev/debugging/tools/#developer-menu) on an `expo-dev-client` app you can do the following:
- Android Device: Shake the device vertically, or if your device is connected via USB, run adb shell input keyevent 82 in your terminal
- Android Emulator: Either press Cmd ⌘ + m or Ctrl + m or run adb shell input keyevent 82 in your terminal
- iOS Device: Shake the device, or touch 3 fingers to the screen
- iOS Simulator: Press Ctrl + Cmd ⌘ + z on a Mac in the emulator to simulate the shake gesture, or press Cmd ⌘ + d
### Running E2E Tests
- Make sure you've setup your environment following above

22
src/lib/sharing.ts 100644
View File

@ -0,0 +1,22 @@
import {isNative} from 'platform/detection'
// import * as Sharing from 'expo-sharing'
import Clipboard from '@react-native-clipboard/clipboard'
import * as Toast from '../view/com/util/Toast'
import {Share} from 'react-native'
/**
* This function shares a URL using the native Share API if available, or copies it to the clipboard
* and displays a toast message if not (mostly on web)
* @param {string} url - A string representing the URL that needs to be shared or copied to the
* clipboard.
*/
export async function shareUrl(url: string) {
if (isNative) {
Share.share({url: url})
} else {
// React Native Share is not supported by web. Web Share API
// has increasing but not full support, so default to clipboard
Clipboard.setString(url)
Toast.show('Copied to clipboard')
}
}

View File

@ -1,7 +1,6 @@
import React from 'react'
import {observer} from 'mobx-react-lite'
import {
Share,
StyleSheet,
TouchableOpacity,
TouchableWithoutFeedback,
@ -31,9 +30,9 @@ import {ProfileHeaderLabels} from '../util/moderation/ProfileHeaderLabels'
import {usePalette} from 'lib/hooks/usePalette'
import {useAnalytics} from 'lib/analytics'
import {NavigationProp} from 'lib/routes/types'
import {isAndroid, isDesktopWeb, isIOS} from 'platform/detection'
import {isDesktopWeb} from 'platform/detection'
import {FollowState} from 'state/models/cache/my-follows'
import Clipboard from '@react-native-clipboard/clipboard'
import {shareUrl} from 'lib/sharing'
const BACK_HITSLOP = {left: 30, top: 30, right: 30, bottom: 30}
@ -152,15 +151,7 @@ const ProfileHeaderLoaded = observer(function ProfileHeaderLoaded({
const onPressShare = React.useCallback(async () => {
track('ProfileHeader:ShareButtonClicked')
const url = toShareUrl(`/profile/${view.handle}`)
if (isIOS || isAndroid) {
Share.share({url})
} else {
// React Native Share is not supported by web. Web Share API
// has increasing but not full support, so default to clipboard
Clipboard.setString(url)
Toast.show('Copied to clipboard')
}
shareUrl(url)
}, [track, view])
const onPressMuteAccount = React.useCallback(async () => {

View File

@ -1,7 +1,6 @@
import React, {useRef} from 'react'
import {
Dimensions,
Share,
StyleProp,
StyleSheet,
TouchableOpacity,
@ -19,10 +18,8 @@ import {toShareUrl} from 'lib/strings/url-helpers'
import {useStores} from 'state/index'
import {usePalette} from 'lib/hooks/usePalette'
import {useTheme} from 'lib/ThemeContext'
import {isAndroid, isIOS} from 'platform/detection'
import Clipboard from '@react-native-clipboard/clipboard'
import * as Toast from '../../util/Toast'
import {isWeb} from 'platform/detection'
import {shareUrl} from 'lib/sharing'
const HITSLOP = {left: 10, top: 10, right: 10, bottom: 10}
const ESTIMATED_BTN_HEIGHT = 50
@ -182,15 +179,7 @@ export function PostDropdownBtn({
label: 'Share...',
onPress() {
const url = toShareUrl(itemHref)
if (isIOS || isAndroid) {
Share.share({url})
} else {
// React Native Share is not supported by web. Web Share API
// has increasing but not full support, so default to clipboard
Clipboard.setString(url)
Toast.show('Copied to clipboard')
}
shareUrl(url)
},
},
{sep: true},