BREAKING CHANGE: Fix skill update after upstream API change

`level` must now be a number instead of allowing for strings, too. Previously the service method `updateSkillByEmail` used `string` instead of `string | number` -> be explicit about requiring `number` there, too.

`saveTo` is now a required key (use previous behaviour of always saving to the profile instead of resumes; value 5).

src/api.ts

@@ -348,8 +348,8 @@ export class Api {
   updateSkill = (
     userId: number | string,
     skillId: number | string,
-    keywordSynonymId: number | string,
-    level: number | string
+    keywordSynonymId: number,
+    level: number
   ) =>
     this.client
       .put(
@@ -358,6 +358,7 @@ export class Api {
           json: {
             keywordSynonymId,
             level,
+            saveTo: 5, // CompanyUserProfileSkillEditModel > AllResumesOfLanguage = 3; Profile = 5;
           },
         }
       )

src/service.ts

@@ -323,7 +323,7 @@ export class CinodeService {
     return { assigned, prospects, openRoles };
   }
 
-  async updateSkillByEmail(email: string, skillName: string, level: string) {
+  async updateSkillByEmail(email: string, skillName: string, level: number) {
     const userId = await this.api.resolveUserIdByEmail(email);
     if (!userId) {
       throw new Error(`No user found with email: ${email}`);