Merge pull request #152 from Scalingo/feat/collaboration/missing-fields

[Collaborator] Add missing fields user_id/app_id to Collaborator model, update semantic of transfer to mention the ID should be given

Author: ksol

src/Apps/index.ts

@@ -130,20 +130,16 @@ export default class Apps {
    * @see https://developers.scalingo.com/apps.html#transfer-ownership-of-an-application
    * @param appID ID of the application
    * @param currentName Current name of the application. Used as validation.
-   * @param ownerEmail Email address of the new owner.
+   * @param ownerID Email address of the new owner.
    * @return Promise that when resolved returns the App transferred.
    */
-  transfer(
-    appID: string,
-    currentName: string,
-    ownerEmail: string
-  ): Promise<App> {
+  transfer(appID: string, currentName: string, ownerID: string): Promise<App> {
     return unpackData(
       this._client.apiClient().patch(
         `/apps/${appID}`,
         {
           app: {
-            owner: ownerEmail,
+            owner: ownerID,
           },
         },
         { params: { current_name: currentName } }

src/models/regional/collaborators.ts

@@ -13,6 +13,10 @@ export interface Collaborator {
   id: string;
   /** Email of the collaborator to invite */
   email: string;
+  /** Unique User ID of the user who accepted the collaboration */
+  user_id: string;
+  /** ID of the application owning the collaborator */
+  app_id: string;
   /** Username of the person to invite */
   username: string;
   /** Status of the invitation */