[server] When a user with no explicit 'usageAttributionId' joins a team with usage-based billing enabled, automatically re-attribute usage to that team

Author: jankeromnes

components/server/ee/src/workspace/gitpod-server-impl.ts

@@ -1428,13 +1428,12 @@ export class GitpodServerEEImpl extends GitpodServerImpl {
 
     protected async onTeamMemberAdded(userId: string, teamId: string): Promise<void> {
         const now = new Date();
-        const teamSubscription = await this.teamSubscription2DB.findForTeam(teamId, now.toISOString());
-        if (!teamSubscription) {
-            // No team subscription, nothing to do 🌴
-            return;
+        const ts2 = await this.teamSubscription2DB.findForTeam(teamId, now.toISOString());
+        if (ts2) {
+            await this.updateTeamSubscriptionQuantity(ts2);
+            await this.teamSubscription2Service.addTeamMemberSubscription(ts2, userId);
         }
-        await this.updateTeamSubscriptionQuantity(teamSubscription);
-        await this.teamSubscription2Service.addTeamMemberSubscription(teamSubscription, userId);
+        // TODO(janx): If Stripe subscription && !user.attributionTeamId --> set user.attributionTeamId to teamId
     }
 
     protected async onTeamMemberRemoved(userId: string, teamId: string, teamMembershipId: string): Promise<void> {