create-personal-chat #15

Merged
Vitalik merged 9 commits from create-personal-chat into stage 9 months ago
Collaborator
There is no content yet.
YaroslavBerkuta added 2 commits 9 months ago
YaroslavBerkuta requested review from Vitalik 9 months ago
YaroslavBerkuta requested review from Developers 9 months ago
Vitalik reviewed 9 months ago
excludeIds: [accountId],
resetOnGoBack: true,
type: 'all',
const onPressHeaderBtn = () => {
Owner

це назва пропса
а назва функції має відображати дію
тобто createChat
або на крайній випадок handlePressCreateChat

це назва пропса а назва функції має відображати дію тобто createChat або на крайній випадок handlePressCreateChat
YaroslavBerkuta marked this conversation as resolved
Vitalik reviewed 9 months ago
import { chatManager } from '@/managers'
import { simpleDispatch } from '@/store/store-helpers'
import { SelectChat } from '@/store/chats'
interface IProps extends IRouteParams {}
Owner

відступи

IProps видалити, параметри навігації брати через хуки

відступи IProps видалити, параметри навігації брати через хуки
YaroslavBerkuta marked this conversation as resolved
Vitalik reviewed 9 months ago
}
const filterList = useMemo(() => {
return items?.filter(it => it.id !== account.id)
Owner

список великий, не хочеться перефільтровувати кожен раз при завантаженні це
Я думаю можна додати параметр до useFetchUsersList щось типу execludeUserId і потім проводити фільтрацію в transformList від useFlatList, щоб фільтрація відбувалась лише один раз

список великий, не хочеться перефільтровувати кожен раз при завантаженні це Я думаю можна додати параметр до useFetchUsersList щось типу execludeUserId і потім проводити фільтрацію в transformList від useFlatList, щоб фільтрація відбувалась лише один раз
YaroslavBerkuta marked this conversation as resolved
Vitalik reviewed 9 months ago
</TouchableOpacity>
)
},
[items],
Owner

забрати залежності, нам не треба змінювати цю функцію якщо завантажився новий список

забрати залежності, нам не треба змінювати цю функцію якщо завантажився новий список
YaroslavBerkuta marked this conversation as resolved
Vitalik reviewed 9 months ago
<FlatList
data={filterList}
renderItem={_renderItem}
contentContainerStyle={{ paddingHorizontal: $size(16) }}
Owner

на всякий випадок flex: 1

на всякий випадок flex: 1
YaroslavBerkuta marked this conversation as resolved
Vitalik reviewed 9 months ago
placeholder={'Знайдіть контакт'}
onChange={setSearchVal}
/>
<FlatList
Owner

список не бескінечний, користувачі не будуть грузитись при скролі.
лоадера нема, немає повідомлення шо список пустий, немає можливості рефрешнути список рухом вниз

список не бескінечний, користувачі не будуть грузитись при скролі. лоадера нема, немає повідомлення шо список пустий, немає можливості рефрешнути список рухом вниз
YaroslavBerkuta marked this conversation as resolved
YaroslavBerkuta added 1 commit 9 months ago
YaroslavBerkuta added 1 commit 9 months ago
Vitalik reviewed 9 months ago
data={items}
renderItem={_renderItem}
contentContainerStyle={{ paddingHorizontal: $size(16) }}
keyExtractor={item => `${item.id}`}
Owner

в функцію з useCallback без залежностей

в функцію з useCallback без залежностей
YaroslavBerkuta marked this conversation as resolved
Vitalik reviewed 9 months ago
Користувачі відсутні
</Txt>
)
else if (true || true)
Owner

Розпиши мені будь-ласка що саме робить ця умова, у відповідь на цей комент

Розпиши мені будь-ласка що саме робить ця умова, у відповідь на цей комент
Poster
Collaborator

нічого перевіряв лоадер, і не змінив умову на ту що має бути

нічого перевіряв лоадер, і не змінив умову на ту що має бути
YaroslavBerkuta marked this conversation as resolved
YaroslavBerkuta added 1 commit 9 months ago
Vitalik reviewed 9 months ago
ios: false,
android: isLoading,
})}
onEndReached={() => loadMore()}
Owner

onEndReached={loadMore}

onEndReached={loadMore}
YaroslavBerkuta marked this conversation as resolved
Vitalik reviewed 9 months ago
android: isLoading,
})}
onEndReached={() => loadMore()}
onRefresh={() => resetFlatList}
Owner

так розумію ти не тестував чи перезавантаження списку працює

так розумію ти не тестував чи перезавантаження списку працює
YaroslavBerkuta marked this conversation as resolved
Vitalik reviewed 9 months ago
}
}
const _renderItem = useCallback(({ item }: { item: IShortUser }) => {
Owner

renderItem

renderItem
YaroslavBerkuta marked this conversation as resolved
YaroslavBerkuta added 1 commit 9 months ago
Vitalik reviewed 9 months ago
Користувачі відсутні
</Txt>
)
else if (isLoading || isLoadingNext)
Owner

видали
попередня умова вже це перевіряє

видали попередня умова вже це перевіряє
YaroslavBerkuta marked this conversation as resolved
YaroslavBerkuta added 1 commit 9 months ago
Vitalik reviewed 9 months ago
showsVerticalScrollIndicator={false}
showsHorizontalScrollIndicator={false}
ListEmptyComponent={() => {
if (!isLoading && !isLoadingNext)
Owner

не весь код видалити треба було
перевірку яка немає сенсу треба було видалити
лоадер треба показувати якщо елементів немає

не весь код видалити треба було перевірку яка немає сенсу треба було видалити лоадер треба показувати якщо елементів немає
Owner

або в renderFooter зробити так щоб лоадер при перезавантаженні також показувався ( тобто коли isLoading = true )

або в renderFooter зробити так щоб лоадер при перезавантаженні також показувався ( тобто коли isLoading = true )
YaroslavBerkuta marked this conversation as resolved
YaroslavBerkuta added 1 commit 9 months ago
Vitalik added 1 commit 9 months ago
Vitalik merged commit 34ac5cb54e into stage 9 months ago
Vitalik deleted branch create-personal-chat 9 months ago
Vitalik referenced this issue from a commit 9 months ago
Vitalik referenced this issue from a commit 9 months ago

Reviewers

Vitalik was requested for review 9 months ago
task-me/Developers was requested for review 9 months ago
The pull request has been merged as 34ac5cb54e.
Sign in to join this conversation.
No reviewers
task-me/Developers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.