fix: try to make sure order when mixed
changes
This commit is contained in:
parent
638e8bf20c
commit
ad2b2993df
@ -345,6 +345,11 @@ function MVPDnD({ user, teams, players }: PlayerInfoProps) {
|
|||||||
handleGet();
|
handleGet();
|
||||||
}, [players]);
|
}, [players]);
|
||||||
|
|
||||||
|
useEffect(() => {
|
||||||
|
handleGet();
|
||||||
|
// setMixedList(rankedPlayers);
|
||||||
|
}, [mixed]);
|
||||||
|
|
||||||
const [dialog, setDialog] = useState("dialog");
|
const [dialog, setDialog] = useState("dialog");
|
||||||
const dialogRef = useRef<HTMLDialogElement>(null);
|
const dialogRef = useRef<HTMLDialogElement>(null);
|
||||||
|
|
||||||
@ -357,6 +362,15 @@ function MVPDnD({ user, teams, players }: PlayerInfoProps) {
|
|||||||
response ? setDialog(response) : setDialog("try sending again");
|
response ? setDialog(response) : setDialog("try sending again");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
const setMixedList = (newList: User[]) =>
|
||||||
|
mixed
|
||||||
|
? setRankedPlayers(
|
||||||
|
newList.sort((a, b) =>
|
||||||
|
a.gender && b.gender ? a.gender.localeCompare(b.gender) : -1
|
||||||
|
)
|
||||||
|
)
|
||||||
|
: setRankedPlayers(newList);
|
||||||
|
|
||||||
async function handleGet() {
|
async function handleGet() {
|
||||||
setLoading(true);
|
setLoading(true);
|
||||||
const data = await apiAuth(`mvps/${teams.activeTeam}`, null, "GET");
|
const data = await apiAuth(`mvps/${teams.activeTeam}`, null, "GET");
|
||||||
@ -366,7 +380,7 @@ function MVPDnD({ user, teams, players }: PlayerInfoProps) {
|
|||||||
setRankedPlayers([]);
|
setRankedPlayers([]);
|
||||||
} else {
|
} else {
|
||||||
const mvps = data as MVPRanking;
|
const mvps = data as MVPRanking;
|
||||||
setRankedPlayers(filterSort(players, mvps.mvps));
|
setMixedList(filterSort(players, mvps.mvps));
|
||||||
setAvailablePlayers(
|
setAvailablePlayers(
|
||||||
players.filter((user) => !mvps.mvps.includes(user.id))
|
players.filter((user) => !mvps.mvps.includes(user.id))
|
||||||
);
|
);
|
||||||
@ -412,17 +426,7 @@ function MVPDnD({ user, teams, players }: PlayerInfoProps) {
|
|||||||
)}
|
)}
|
||||||
<PlayerList
|
<PlayerList
|
||||||
list={rankedPlayers}
|
list={rankedPlayers}
|
||||||
setList={(newList) =>
|
setList={setMixedList}
|
||||||
mixed
|
|
||||||
? setRankedPlayers(
|
|
||||||
newList.sort((a, b) =>
|
|
||||||
a.gender && b.gender
|
|
||||||
? a.gender.localeCompare(b.gender)
|
|
||||||
: -1
|
|
||||||
)
|
|
||||||
)
|
|
||||||
: setRankedPlayers(newList)
|
|
||||||
}
|
|
||||||
group={{
|
group={{
|
||||||
name: "mvp-shared",
|
name: "mvp-shared",
|
||||||
}}
|
}}
|
||||||
|
Loading…
x
Reference in New Issue
Block a user