diff options
author | Leon Jiang <35908040+leonyjiang@users.noreply.github.com> | 2021-03-15 18:54:10 -0400 |
---|---|---|
committer | Leon Jiang <35908040+leonyjiang@users.noreply.github.com> | 2021-03-15 18:54:10 -0400 |
commit | a544272fdead4a351f9750180587c6d4e7e1cecc (patch) | |
tree | cde9e9b875ce7ca8e769bc8a0fbfe1351308fe20 /src | |
parent | da60a2a3921a96a5c0b26f682f1b7d5c971dd6df (diff) |
Update query to get randomized ordering
Diffstat (limited to 'src')
-rw-r--r-- | src/screens/suggestedPeople/SuggestedPeopleScreen.tsx | 12 | ||||
-rw-r--r-- | src/services/SuggestedPeopleService.ts | 8 |
2 files changed, 15 insertions, 5 deletions
diff --git a/src/screens/suggestedPeople/SuggestedPeopleScreen.tsx b/src/screens/suggestedPeople/SuggestedPeopleScreen.tsx index 61adc171..651d858c 100644 --- a/src/screens/suggestedPeople/SuggestedPeopleScreen.tsx +++ b/src/screens/suggestedPeople/SuggestedPeopleScreen.tsx @@ -8,7 +8,6 @@ import { ViewToken, } from 'react-native'; -import Animated from 'react-native-reanimated'; import {useDispatch, useSelector, useStore} from 'react-redux'; import {TabsGradient, TaggLoadingIndicator, Background} from '../../components'; import {SP_PAGE_SIZE} from '../../constants'; @@ -39,7 +38,6 @@ import SPBody from './SPBody'; */ const SuggestedPeopleScreen: React.FC = () => { - const y = Animated.useValue(0); const navigation = useNavigation(); const state: RootState = useStore().getState(); const dispatch = useDispatch(); @@ -65,6 +63,11 @@ const SuggestedPeopleScreen: React.FC = () => { people.length ? setLoading(false) : setLoading(true); }, [people]); + /* + * The randomly-generated seed used by the API to order the suggested people. + */ + const randomizationSeed = Math.random(); + const stausBarRef = useRef(hideStatusBar); // loads data and append it to users based on current page @@ -106,6 +109,7 @@ const SuggestedPeopleScreen: React.FC = () => { await getSuggestedPeople( SP_PAGE_SIZE, shouldResetData ? 0 : page * SP_PAGE_SIZE, + randomizationSeed, ); loadNextPage().then((newUsers) => { @@ -131,9 +135,11 @@ const SuggestedPeopleScreen: React.FC = () => { await Promise.all(users.map((user) => loadUserData(user))); }; + // TODO: only reload data, don't reset screen type, which causes the entire + // screen to re-render and makes for bad UX const onRefresh = () => { const reset = async () => { - await dispatch(resetScreenType(screenType)); + dispatch(resetScreenType(screenType)); setShouldResetData(true); }; setRefreshing(true); diff --git a/src/services/SuggestedPeopleService.ts b/src/services/SuggestedPeopleService.ts index e51914b0..4f56feb9 100644 --- a/src/services/SuggestedPeopleService.ts +++ b/src/services/SuggestedPeopleService.ts @@ -63,10 +63,14 @@ export const sendSuggestedPeoplePhoto = async (photoUri: string) => { } }; -export const getSuggestedPeople = async (limit: number, offset: number) => { +export const getSuggestedPeople = async ( + limit: number, + offset: number, + seed: number, +) => { try { const token = await AsyncStorage.getItem('token'); - const url = `${SP_USERS_ENDPOINT}?limit=${limit}&offset=${offset}`; + const url = `${SP_USERS_ENDPOINT}?limit=${limit}&offset=${offset}&seed=${seed}`; const response = await fetch(url, { method: 'GET', headers: { |