update profile picture (danny-avila#792)

bsu3338 · Aug 17, 2023 · 1dacfa4 · 1dacfa4
1 parent afd43af
commit 1dacfa4
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 5 deletions.
diff --git a/api/strategies/discordStrategy.js b/api/strategies/discordStrategy.js
@@ -17,11 +17,6 @@ const discordLogin = async () =>
         const email = profile.email;
         const discordId = profile.id;
 
-        const oldUser = await User.findOne({ email });
-        if (oldUser) {
-          return cb(null, oldUser);
-        }
-
         let avatarURL;
         if (profile.avatar) {
           const format = profile.avatar.startsWith('a_') ? 'gif' : 'png';
@@ -31,6 +26,13 @@ const discordLogin = async () =>
           avatarURL = `https://cdn.discordapp.com/embed/avatars/${defaultAvatarNum}.png`;
         }
 
+        const oldUser = await User.findOne({ email });
+        if (oldUser) {
+          oldUser.avatar = avatarURL;
+          await oldUser.save();
+          return cb(null, oldUser);
+        }
+
         const newUser = await User.create({
           provider: 'discord',
           discordId,

diff --git a/api/strategies/githubStrategy.js b/api/strategies/githubStrategy.js
@@ -23,6 +23,8 @@ const githubLogin = async () =>
 
         const oldUser = await User.findOne({ email });
         if (oldUser) {
+          oldUser.avatar = profile.photos[0].value;
+          await oldUser.save();
           return cb(null, oldUser);
         }
 

diff --git a/api/strategies/googleStrategy.js b/api/strategies/googleStrategy.js
@@ -17,6 +17,8 @@ const googleLogin = async () =>
       try {
         const oldUser = await User.findOne({ email: profile.emails[0].value });
         if (oldUser) {
+          oldUser.avatar = profile.photos[0].value;
+          await oldUser.save();
           return cb(null, oldUser);
         }
       } catch (err) {