brandon.presley/Massive
brandon.presley
/
Massive
4
8
Fork 3
Code Issues 14 Pull Requests Activity

Reduce state in SetList 

The `set` state here was just a copy of the first element
of `sets`.
This commit is contained in:
Brandon Presley 2022-12-18 12:48:20 +13:00
parent f02249e254
commit 888ae576b0
1 changed files with 6 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
SetList.tsx
View File

@ -19,7 +19,6 @@ const limit = 15
export default function SetList() { export default function SetList() {
const [sets, setSets] = useState<GymSet[]>([]) const [sets, setSets] = useState<GymSet[]>([])
const [set, setSet] = useState<GymSet>(defaultSet)
const [offset, setOffset] = useState(0) const [offset, setOffset] = useState(0)
const [term, setTerm] = useState('') const [term, setTerm] = useState('')
const [end, setEnd] = useState(false) const [end, setEnd] = useState(false)
@ -38,11 +37,6 @@ export default function SetList() {
setSets(newSets) setSets(newSets)
setOffset(0) setOffset(0)
setEnd(false) setEnd(false)
const first = newSets[0]
if (!first) return
const newSet = {...first}
delete newSet.id
setSet(newSet)
}, []) }, [])
useFocusEffect( useFocusEffect(
@ -84,15 +78,13 @@ export default function SetList() {
}, [term, end, offset, sets]) }, [term, end, offset, sets])
const onAdd = useCallback(async () => { const onAdd = useCallback(async () => {
console.log(`${SetList.name}.onAdd`, {set})
const [{now}] = await getNow() const [{now}] = await getNow()
const newSet: GymSet = set || { let set = sets[0]
...defaultSet, if (!set) set = {...defaultSet}
created: now, set.created = now
} delete set.id
newSet.created = now navigation.navigate('EditSet', {set})
navigation.navigate('EditSet', {set: newSet}) }, [navigation, sets])
}, [navigation, set])
const search = useCallback( const search = useCallback(
(value: string) => { (value: string) => {