Merge pull request #63 from zhumeisongsong/feature/new-UserDto-in-Res…

…olver fix: 🐛 new UserDto when getUser in Resolver
zhumeisongsong · Nov 20, 2024 · 220cf0b · 220cf0b
2 parents 13a0728 + fbb2630
commit 220cf0b
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/libs/users/interface-adapters/src/lib/resolver/users.resolver.ts b/libs/users/interface-adapters/src/lib/resolver/users.resolver.ts
@@ -8,7 +8,13 @@ export class UsersResolver {
   constructor(private usersService: UsersService) {}
 
   @Query(() => UserDto, { nullable: true })
-  getUser(@Args({ name: 'id', type: () => ID }) id: string): Promise<UserDto | null> {
-    return this.usersService.findById(id);
+  async getUser(@Args({ name: 'id', type: () => ID }) id: string): Promise<UserDto | null> {
+    const user = await this.usersService.findById(id);
+
+    if (!user) {
+      return null;
+    }
+
+    return new UserDto(user.id, user.name);
   }
 }