mirror of
https://github.com/Steffo99/sophon.git
synced 2024-12-22 14:54:22 +00:00
🐛 Fix rerenders on every location change
This commit is contained in:
parent
7ebc8a82d8
commit
5b62f534f0
4 changed files with 76 additions and 58 deletions
|
@ -28,7 +28,8 @@ import {ThemeProvider} from "./contexts/theme"
|
||||||
|
|
||||||
|
|
||||||
function App({..._}: RouteComponentProps) {
|
function App({..._}: RouteComponentProps) {
|
||||||
return (
|
return React.useMemo(
|
||||||
|
() => (
|
||||||
<InstanceProvider>
|
<InstanceProvider>
|
||||||
<InstanceRouter
|
<InstanceRouter
|
||||||
unselectedRoute={() => <>
|
unselectedRoute={() => <>
|
||||||
|
@ -37,12 +38,12 @@ function App({..._}: RouteComponentProps) {
|
||||||
selectedRoute={() => <>
|
selectedRoute={() => <>
|
||||||
<InstanceDescriptionBox/>
|
<InstanceDescriptionBox/>
|
||||||
<AuthorizationProvider>
|
<AuthorizationProvider>
|
||||||
|
<CacheProvider>
|
||||||
<AuthorizationRouter
|
<AuthorizationRouter
|
||||||
unselectedRoute={() => <>
|
unselectedRoute={() => <>
|
||||||
<AuthorizationStepPage/>
|
<AuthorizationStepPage/>
|
||||||
</>}
|
</>}
|
||||||
selectedRoute={() => <>
|
selectedRoute={() => <>
|
||||||
<CacheProvider>
|
|
||||||
<GroupRouter
|
<GroupRouter
|
||||||
unselectedRoute={({viewSet}) => <>
|
unselectedRoute={({viewSet}) => <>
|
||||||
<GroupListBox viewSet={viewSet}/>
|
<GroupListBox viewSet={viewSet}/>
|
||||||
|
@ -69,13 +70,15 @@ function App({..._}: RouteComponentProps) {
|
||||||
/>
|
/>
|
||||||
</>}
|
</>}
|
||||||
/>
|
/>
|
||||||
</CacheProvider>
|
|
||||||
</>}
|
</>}
|
||||||
/>
|
/>
|
||||||
|
</CacheProvider>
|
||||||
</AuthorizationProvider>
|
</AuthorizationProvider>
|
||||||
</>}
|
</>}
|
||||||
/>
|
/>
|
||||||
</InstanceProvider>
|
</InstanceProvider>
|
||||||
|
),
|
||||||
|
[],
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -2,7 +2,6 @@ import {Box, Details, Form, Idiomatic as I, useFormState} from "@steffo/bluelib-
|
||||||
import * as React from "react"
|
import * as React from "react"
|
||||||
import {useAuthorizationContext} from "../../contexts/authorization"
|
import {useAuthorizationContext} from "../../contexts/authorization"
|
||||||
import {useCacheContext} from "../../contexts/cache"
|
import {useCacheContext} from "../../contexts/cache"
|
||||||
import {useInstanceContext} from "../../contexts/instance"
|
|
||||||
import {ManagedResource, ManagedViewSet} from "../../hooks/useManagedViewSet"
|
import {ManagedResource, ManagedViewSet} from "../../hooks/useManagedViewSet"
|
||||||
import {SophonResearchGroup} from "../../types/SophonTypes"
|
import {SophonResearchGroup} from "../../types/SophonTypes"
|
||||||
|
|
||||||
|
@ -14,7 +13,6 @@ export interface GroupCreateBoxProps {
|
||||||
|
|
||||||
|
|
||||||
export function GroupCreateBox({viewSet, resource}: GroupCreateBoxProps): JSX.Element | null {
|
export function GroupCreateBox({viewSet, resource}: GroupCreateBoxProps): JSX.Element | null {
|
||||||
const instance = useInstanceContext()
|
|
||||||
const authorization = useAuthorizationContext()
|
const authorization = useAuthorizationContext()
|
||||||
const cache = useCacheContext()
|
const cache = useCacheContext()
|
||||||
|
|
||||||
|
@ -33,7 +31,7 @@ export function GroupCreateBox({viewSet, resource}: GroupCreateBoxProps): JSX.El
|
||||||
}).reduce((a, b) => {
|
}).reduce((a, b) => {
|
||||||
return {...a, ...b}
|
return {...a, ...b}
|
||||||
}),
|
}),
|
||||||
[instance, authorization],
|
[authorization, cache],
|
||||||
)
|
)
|
||||||
|
|
||||||
const members =
|
const members =
|
||||||
|
@ -83,7 +81,7 @@ export function GroupCreateBox({viewSet, resource}: GroupCreateBoxProps): JSX.El
|
||||||
(
|
(
|
||||||
authorization && authorization.state.user && trueMembers?.includes(authorization.state.user.id)
|
authorization && authorization.state.user && trueMembers?.includes(authorization.state.user.id)
|
||||||
),
|
),
|
||||||
[authorization, resource],
|
[authorization, resource, trueMembers],
|
||||||
)
|
)
|
||||||
|
|
||||||
const canAdministrate =
|
const canAdministrate =
|
||||||
|
|
|
@ -22,17 +22,27 @@ export function GroupMembersBox({resource}: GroupMembersBoxProps): JSX.Element |
|
||||||
|
|
||||||
const trueMembers = [...new Set([resource.value.owner, ...resource.value.members])]
|
const trueMembers = [...new Set([resource.value.owner, ...resource.value.members])]
|
||||||
|
|
||||||
|
const users = trueMembers.map((id, index) => {
|
||||||
|
const user = cache.getUserById(id)
|
||||||
|
|
||||||
|
if(!user) {
|
||||||
|
return null
|
||||||
|
}
|
||||||
|
|
||||||
|
return (
|
||||||
|
<ListUnordered.Item bluelibClassNames={index === 0 ? "color-blue" : ""} key={id}>
|
||||||
|
{user.value.username}
|
||||||
|
</ListUnordered.Item>
|
||||||
|
)
|
||||||
|
})
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<Box>
|
<Box>
|
||||||
<Heading level={3}>
|
<Heading level={3}>
|
||||||
Members of <Idiomatic>{resource.value.name}</Idiomatic>
|
Members of <Idiomatic>{resource.value.name}</Idiomatic>
|
||||||
</Heading>
|
</Heading>
|
||||||
<ListUnordered>
|
<ListUnordered>
|
||||||
{trueMembers.map((id, index) => (
|
{users}
|
||||||
<ListUnordered.Item bluelibClassNames={index === 0 ? "color-blue" : ""} key={id}>
|
|
||||||
{cache.getUserById(id)!.value.username}
|
|
||||||
</ListUnordered.Item>
|
|
||||||
))}
|
|
||||||
</ListUnordered>
|
</ListUnordered>
|
||||||
</Box>
|
</Box>
|
||||||
)
|
)
|
||||||
|
|
|
@ -586,28 +586,35 @@ export function useManagedViewSet<Resource extends DjangoResource>(baseRoute: st
|
||||||
useEffect(
|
useEffect(
|
||||||
() => {
|
() => {
|
||||||
if(!refreshOnMount) {
|
if(!refreshOnMount) {
|
||||||
|
console.debug("[ManagedViewSet |", baseRoute, "] Not refreshing: refreshOnMount is", refreshOnMount)
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
if(!viewset) {
|
if(!viewset) {
|
||||||
|
console.debug("[ManagedViewSet |", baseRoute, "] Not refreshing: viewset could not be initialized")
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
if(!state.firstRun) {
|
if(!state.firstRun) {
|
||||||
|
console.debug("[ManagedViewSet |", baseRoute, "] Not refreshing: this is not the first run")
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
if(state.busy) {
|
if(state.busy) {
|
||||||
|
console.debug("[ManagedViewSet |", baseRoute, "] Not refreshing: viewset is busy")
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
if(state.error) {
|
if(state.error) {
|
||||||
|
console.debug("[ManagedViewSet |", baseRoute, "] Not refreshing: an error occoured")
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
if(state.resources) {
|
if(state.resources) {
|
||||||
|
console.debug("[ManagedViewSet |", baseRoute, "] Not refreshing: resources are already available")
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
// noinspection JSIgnoredPromiseFromCall
|
// noinspection JSIgnoredPromiseFromCall
|
||||||
refresh()
|
refresh()
|
||||||
|
console.debug("[ManagedViewSet |", baseRoute, "] Requested a refresh successfully")
|
||||||
},
|
},
|
||||||
[refresh, state, refreshOnMount],
|
[refresh, state, refreshOnMount, viewset, baseRoute],
|
||||||
)
|
)
|
||||||
|
|
||||||
if(!viewset) {
|
if(!viewset) {
|
||||||
|
|
Loading…
Reference in a new issue