Merge 2d8ae3e301 into b9cbd610a9

Bumps [y18n](https://github.com/yargs/y18n) from 3.2.1 to 3.2.2.
- [Release notes](https://github.com/yargs/y18n/releases)
- [Changelog](https://github.com/yargs/y18n/blob/master/CHANGELOG.md)
- [Commits](https://github.com/yargs/y18n/commits)

Signed-off-by: dependabot[bot] <support@github.com>

README.md

@@ -1,2 +1,2 @@
 # CS307-Team24
-CS307 Team 24 Twistter website.
+CS307 Team 24 Twistter website

functions/handlers/post.js

@@ -178,10 +178,10 @@ exports.getOtherUsersPosts = (req, res) => {
     .collection("posts")
     .where("userHandle", "==", req.body.handle);
     
-    post_query += admin
+    // post_query += admin
-    .firestore()
+    // .firestore()
-    .collection("posts")
+    // .collection("posts")
-    .where("microBlogTitle", "==", "Alert").where("userHandle", "==", "Admin");
+    // .where("microBlogTitle", "==", "Alert").where("userHandle", "==", "Admin");
 
   post_query
     .get()

functions/handlers/users.js

@@ -1,5 +1,6 @@
 /* eslint-disable promise/catch-or-return */
 /* eslint-disable promise/always-return */
+/* eslint-disable prefer-promise-reject-error */
 
 const { admin, db } = require("../util/admin");
 const config = require("../util/config");
@@ -866,10 +867,10 @@ oneWayCheck = (userA, userB) => {
             if (dmRecipient === userB) {
               console.log(`You already have a DM with ${userB}`);
               // reject(new Error(`You already have a DM with ${userB}`));
-              reject({
+              let e = new Error(`You already have a DM with that user`);
-                code: 400,
+              e.code = 400,
-                message: `You already have a DM with that user`
+              e.message = `You already have a DM with that user`
-              });
+              reject(e);
               return;
             }
           });
@@ -1212,7 +1213,10 @@ exports.addSubscription = (req, res) => {
         return res.status(500).json({ err });
       });
     // return res.status(200).json({ message: "ok" });
-  });
+  })
+  .catch((error) => {
+    return res.status(400).json({message: "That user doesn't exist", error});
+  })
 };
 
 exports.getSubs = (req, res) => {

functions/package-lock.json

@@ -673,6 +673,14 @@
         "readable-stream": "~1.0.32"
       }
     },
+    "busboy": {
+      "version": "0.3.1",
+      "resolved": "https://registry.npmjs.org/busboy/-/busboy-0.3.1.tgz",
+      "integrity": "sha512-y7tTxhGKXcyBxRKAni+awqx8uqaJKrSFSNFSeRG5CsWNdmy2BIK+6VGWEW7TZnIO/533mtMEA4rOevQV815YJw==",
+      "requires": {
+        "dicer": "0.3.0"
+      }
+    },
     "bytebuffer": {
       "version": "5.0.1",
       "resolved": "https://registry.npmjs.org/bytebuffer/-/bytebuffer-5.0.1.tgz",
@@ -3707,9 +3715,9 @@
       "integrity": "sha512-LKYU1iAXJXUgAXn9URjiu+MWhyUXHsvfp7mcuYm9dSUKK0/CjtrUwFAxD82/mCWbtLsGjFIad0wIsod4zrTAEQ=="
     },
     "y18n": {
-      "version": "3.2.1",
+      "version": "3.2.2",
-      "resolved": "https://registry.npmjs.org/y18n/-/y18n-3.2.1.tgz",
+      "resolved": "https://registry.npmjs.org/y18n/-/y18n-3.2.2.tgz",
-      "integrity": "sha1-bRX7qITAhnnA136I53WegR4H+kE="
+      "integrity": "sha512-uGZHXkHnhF0XeeAPgnKfPv1bgKAYyVvmNL1xlKsPYZPaIHxGti2hHqvOCQv71XMsLxu1QjergkqogUnms5D3YQ=="
     },
     "yallist": {
       "version": "3.0.3",

twistter-frontend/src/Writing_Microblogs.js

@@ -84,23 +84,23 @@ class Writing_Microblogs extends Component {
         console.error(err);
       });
     console.log(postData.microBlogTopics);
-    let topicPromises = [];
+    // let topicPromises = [];
-    postData.microBlogTopics.forEach(topic => {
+    // postData.microBlogTopics.forEach(topic => {
-      topicPromises.push(axios
+    //   topicPromises.push(axios
-        .post("/putTopic", {
+    //     .post("/putTopic", {
-          following: topic
+    //       following: topic
-        })
+    //     })
-        .then(res => {
+    //     .then(res => {
-          console.log(res.data);
+    //       console.log(res.data);
-        })
+    //     })
-        .catch(err => {
+    //     .catch(err => {
-          console.error(err);
+    //       console.error(err);
-        })
+    //     })
-      )
+    //   )
-    });
+    // });
     event.preventDefault();
-    topicPromises.push(postPromise);
+    // topicPromises.push(postPromise);
-    Promise.all(topicPromises)
+    Promise.all([postPromise])
       .then(() => {
         this.setState({ 
           value: "", 

twistter-frontend/src/pages/Home.js

@@ -57,14 +57,28 @@ class Home extends Component {
       })
       .catch(err => console.log(err));
 
+    let allPosts;
     let postPromise = axios
       .get("/getallPosts")
       .then(res => {
         // console.log(res.data);
+        // this.setState({
+        //   posts: res.data
+        // });
+        allPosts = res.data;
+        // console.log(allPosts)
+        return axios.get("/getAlert")
+      })
+      .then((res) => {
+        // console.log(res.data)
+        // res.data.forEach((adminAlert) => {
+        //   allPosts.push(adminAlert);
+        // })
         this.setState({
-          posts: res.data
+          posts: allPosts
         });
       })
+
       .catch(err => console.log(err));
 
     Promise.all([userPromise, postPromise])
@@ -140,9 +154,8 @@ class Home extends Component {
     }
 
     console.log(hiddenBool);
-    let postMarkup = this.state.posts ? (
+    let postMarkup = this.state.posts ? ( this.state.following === undefined || this.state.following === null ? <Typography>You aren't following anybody right now</Typography> :
-// <<<<<<< admin-delete
+      this.state.posts.map(post => !post.hidden && this.state.following && (this.state.following.includes(post.userHandle) || post.userHandle === "Admin") ? (
-      this.state.posts.map(post => !post.hidden && this.state.following && this.state.following.includes(post.userHandle) ? (
             <Card className={classes.card} key={post.postId}>
               <CardContent>
                 <Typography>
@@ -191,7 +204,7 @@ class Home extends Component {
 
                 {/* <button>Quote</button> */}
 
-                <Typography variant="body2" color={"textSecondary"}>Likes {post.likeCount} Comments {post.commentCount}</Typography>
+                {/* <Typography variant="body2" color={"textSecondary"}>Likes {post.likeCount} Comments {post.commentCount}</Typography> */}
                 
               </CardContent>
             </Card>
@@ -199,18 +212,6 @@ class Home extends Component {
               <p></p>
             )
     )
-// =======
-//       this.state.posts.map(post =>
-//         this.state.following ? (
-//           this.state.following.includes(post.userHandle) ? (
-//           ) : (
-//             <p></p>
-//           )
-//         ) : (
-//           <p></p>
-//         )
-// >>>>>>> master
-      
     ) : (
       <p>Loading post...</p>
     );

twistter-frontend/src/pages/directMessages.js

@@ -83,6 +83,10 @@ const styles = {
 		wordBreak: "break-all",
 		color: 'black'
 	},
+	dmRecentMessageDisabled: {
+		wordBreak: "break-all",
+		color: 'red'
+	},
 	dmListItemContainer: {
 		height: 100
 	},
@@ -105,7 +109,7 @@ const styles = {
 		fontSize: 20,
 		backgroundColor: '#1da1f2',
 		width: 300
-	},
+  },
 	messagesGrid: {
 		// // margin: "auto"
 		// height: "auto",
@@ -426,9 +430,15 @@ export class directMessages extends Component {
 								<Typography
 									className={
 										this.state.selectedChannel && this.state.selectedChannel.dmId === channel.dmId ? (
-											classes.dmRecentMessageSelected
+                      channel.hasDirectMessagesEnabled ? 
+                        classes.dmRecentMessageSelected
+                      :
+                        classes.dmRecentMessageDisabled
 										) : (
-											classes.dmRecentMessageUnselected
+                      channel.hasDirectMessagesEnabled ?
+                        classes.dmRecentMessageUnselected
+                      :
+                        classes.dmRecentMessageDisabled
 										)
 									}
 								>

twistter-frontend/src/pages/otherUser.js

@@ -312,7 +312,7 @@ class user extends Component {
             <Typography variant="body2">{post.body}</Typography>
             <br />
             <Typography variant="body2">
-              <b>Topics:</b> {post.microBlogTopics}
+              <b>Topics:</b> {post.microBlogTopics.join(", ")}
             </Typography>
             <br />
             <Typography variant="body2" color={"textSecondary"}>

twistter-frontend/src/pages/user.js

@@ -269,6 +269,15 @@ class user extends Component {
       </Link>
     ) : null;
 
+    let verifyButtonMarkup = this.state.profile === "Admin" ?
+      <Link to="/verify">
+        <Button className={classes.button} variant="outlined" color="primary">
+          Verify Users
+        </Button>
+      </Link>
+    :
+      null
+
     return (
       this.state.loading ? <CircularProgress size={60} style={{marginTop: "300px"}}></CircularProgress> :
       <div>
@@ -278,6 +287,7 @@ class user extends Component {
             <Grid container>
               <Grid item sm>
                 {editButtonMarkup}
+                {verifyButtonMarkup}
               </Grid>
               <Grid item sm>
                 {/* <Grid container direction="column"> */}

twistter-frontend/src/redux/actions/userActions.js

@@ -36,6 +36,7 @@ export const getUserData = () => (dispatch) => {
 
 // Sends login data to firebase and sets the user data in Redux
 export const loginUser = (loginData, history) => (dispatch) => {
+    dispatch({type: CLEAR_ERRORS});
     dispatch({ type: LOADING_UI });
     axios
       .post("/login", loginData)
@@ -57,6 +58,7 @@ export const loginUser = (loginData, history) => (dispatch) => {
 
 // Sends signup data to firebase and sets the user data in Redux
 export const signupUser = (newUserData, history) => (dispatch) => {
+    dispatch({type: CLEAR_ERRORS});
     dispatch({ type: LOADING_UI });
     axios
       .post("/signup", newUserData)