dave/eveship.fit
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: skills not mentioned in ESI are set to level 1 (instead of level 0) (#21)

Browse Source 
This is because ESI only returns skills that are learnt, and
dogma-engine sets unmentioned skills to level 1.
This commit is contained in:
Patric Stout
2023-11-27 19:33:00 +01:00
committed by GitHub
parent f9d801a1dc
commit c897cb02ba
1 changed files with 9 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
components/Skills/Skills.tsx
View File

@@ -31,10 +31,16 @@ export const Skills = ({ setSkills }: { setSkills: (skills: Record<string, numbe
return;
}
const skills = esi.characters[skillSet]?.skills;
if (!skills) return;
const esiSkills = esi.characters[skillSet]?.skills;
if (!esiSkills) return;
setSkills(skills);
/* Ensure that all skills not mentioned in ESI are set to level 0. */
const newSkills: Record<string, number> = {};
for (const typeId in eveData.typeIDs) {
if (eveData?.typeIDs?.[typeId].categoryID !== 16) continue;
newSkills[typeId] = esiSkills[typeId] ?? 0;
}
setSkills(newSkills);
}, [eveData, esi.characters, setSkills, skillSet]);
React.useEffect(() => {