Skip to content

Commit

Permalink
Googleアカウントでログインした場合は初回のみアカウント情報を登録するように変更
Browse files Browse the repository at this point in the history
  • Loading branch information
MurakawaTakuya committed Nov 21, 2024
1 parent 5dbc189 commit 7a86bfc
Show file tree
Hide file tree
Showing 5 changed files with 23 additions and 24 deletions.
12 changes: 6 additions & 6 deletions functions/src/routers/userRouter.ts
Original file line number Diff line number Diff line change
Expand Up @@ -90,13 +90,13 @@ router.route("/").post(async (req: Request, res: Response) => {
}

// 既に同じ名前のuserが存在する場合はエラーを返す
const userSnapshot = await getUserFromName(name);
// const userSnapshot = await getUserFromName(name);

if (!userSnapshot.empty) {
return res.status(409).json({
message: `A user with the same user name '${name}' already exists`,
});
}
// if (!userSnapshot.empty) {
// return res.status(409).json({
// message: `A user with the same user name '${name}' already exists`,
// });
// }

try {
// userIdをドキュメント名として使用してデータを保存
Expand Down
5 changes: 2 additions & 3 deletions src/utils/createUserAuth.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,7 @@ export const createUser = (email: string, password: string, name: string) => {
// TODO: ログイン状態の維持
})
.catch((error) => {
const errorCode = error.code ?? "unknown";
const errorMessage = error.message;
throw { errorCode, errorMessage };
console.error("errorCode:", (error as any)?.errorCode);
console.error("errorMessage:", (error as any)?.errorMessage);
});
};
5 changes: 2 additions & 3 deletions src/utils/loginUserAuth.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,8 @@ export const loginUser = (email: string, password: string) => {
console.log(user.uid, userData);
})
.catch((error) => {
const errorCode = error.code;
const errorMessage = error.message;
throw new Error(`Error ${errorCode}: ${errorMessage}`);
console.error("errorCode:", (error as any)?.errorCode);
console.error("errorMessage:", (error as any)?.errorMessage);
});
return null;
};
7 changes: 3 additions & 4 deletions src/utils/signInAnonymously.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,10 @@ import { signInAnonymously } from "firebase/auth";
export const signInAsGuest = async () => {
try {
const result = await signInAnonymously(auth);
console.log("Guest login successful:", result.user);
// console.log("Guest login successful:", result.user);
console.log("Guest login successful");
} catch (error) {
const errorCode = (error as any).code;
const errorMessage = (error as any).message;
throw new Error(`Error ${errorCode}: ${errorMessage}`);
console.error("errorCode:", (error as any)?.errorCode);
console.error("errorMessage:", (error as any)?.errorMessage);
}
};
18 changes: 10 additions & 8 deletions src/utils/signInWithGoogleAccount.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { auth, googleProvider } from "@/app/firebase";
import { signInWithPopup } from "firebase/auth";
import { getAdditionalUserInfo, signInWithPopup } from "firebase/auth";
import { createUserAPI } from "./createUserAPI";

/**
Expand All @@ -10,16 +10,18 @@ export const signInWithGoogleAccount = async () => {
try {
const result = await signInWithPopup(auth, googleProvider);

await createUserAPI(
result.user.displayName ?? "Anonymous",
result.user.uid
);
// 初めての時だけユーザー情報を登録する
if (getAdditionalUserInfo(result)?.isNewUser) {
await createUserAPI(
result.user.displayName ?? "no name",
result.user.uid
);
}

console.log("Google login successful");
// console.log("Google login successful:", result.user);
} catch (error) {
const errorCode = (error as any).code;
const errorMessage = (error as any).message;
throw new Error(`Error ${errorCode}: ${errorMessage}`);
console.error("errorCode:", (error as any)?.errorCode);
console.error("errorMessage:", (error as any)?.errorMessage);
}
};

0 comments on commit 7a86bfc

Please sign in to comment.