diff --git a/functions/handlers/topic.js b/functions/handlers/topic.js index bb5b9f1..88e014e 100644 --- a/functions/handlers/topic.js +++ b/functions/handlers/topic.js @@ -1,20 +1,28 @@ const { admin, db } = require("../util/admin"); exports.putTopic = (req, res) => { - const newTopic = { - topic: req.body.topic - }; - - admin - .firestore() - .collection("topics") - .add(newTopic) + let new_following = []; + let userRef = db.doc(`/users/${req.userData.handle}`); + userRef + .get() .then(doc => { - const resTopic = newTopic; - return res.status(200).json(resTopic); + new_following = doc.data().followedTopics; + new_following.push(req.body.following); + + // add stuff + userRef + .set({ followedTopics: new_following }, { merge: true }) + .then(doc => { + return res + .status(201) + .json({ message: `Following ${req.body.following}` }); + }) + .catch(err => { + return res.status(500).json({ err }); + }); + return res.status(200).json({ message: "OK" }); }) .catch(err => { - console.error(err); - return res.status(500).json({ error: "something is wrong" }); + return res.status(500).json({ err }); }); }; @@ -40,22 +48,34 @@ exports.getAllTopics = (req, res) => { }; exports.deleteTopic = (req, res) => { - const topic = db.doc(`/topics/${req.params.topicId}`); - topic + let new_following = []; + let userRef = db.doc(`/users/${req.userData.handle}`); + userRef .get() .then(doc => { - if (!doc.exists) { - return res.status(404).json({ error: "Topic not found" }); - } else { - return topic.delete(); - } - }) - .then(() => { - return res.json({ message: "Topic successfully deleted!" }); + new_following = doc.data().followedTopics; + // remove username from array + new_following.forEach(function(follower, index) { + if (follower === `${req.body.unfollow}`) { + new_following.splice(index, 1); + } + }); + + // update database + userRef + .set({ followedTopics: new_following }, { merge: true }) + .then(doc => { + return res + .status(202) + .json({ message: `Successfully unfollow ${req.body.unfollow}` }); + }) + .catch(err => { + return res.status(500).json({ err }); + }); + return res.status(200).json({ message: "ok" }); }) .catch(err => { - console.error(err); - return res.status(500).json({ error: "Failed to delete topic." }); + return res.status(500).json({ err }); }); }; diff --git a/functions/index.js b/functions/index.js index df44c82..4626e47 100644 --- a/functions/index.js +++ b/functions/index.js @@ -96,7 +96,7 @@ app.post("/putTopic", fbAuth, putTopic); app.get("/getAllTopics", fbAuth, getAllTopics); // delete a specific topic -app.delete("/deleteTopic/:topicId", fbAuth, deleteTopic); +app.post("/deleteTopic", fbAuth, deleteTopic); // get topic for this user app.post("/getUserTopics", fbAuth, getUserTopics); diff --git a/twistter-frontend/src/pages/user.js b/twistter-frontend/src/pages/user.js index 750fc0a..9896f7d 100644 --- a/twistter-frontend/src/pages/user.js +++ b/twistter-frontend/src/pages/user.js @@ -1,290 +1,301 @@ /* eslint-disable */ -import React, { Component } from 'react'; -import PropTypes from 'prop-types'; -import { connect } from 'react-redux'; -import axios from 'axios'; +import React, { Component } from "react"; +import PropTypes from "prop-types"; +import { connect } from "react-redux"; +import axios from "axios"; //import '../App.css'; // Material-UI -import withStyles from '@material-ui/core/styles/withStyles'; -import { makeStyles, styled } from '@material-ui/core/styles'; -import { Link } from 'react-router-dom'; -import Card from '@material-ui/core/Card'; -import CardMedia from '@material-ui/core/CardMedia'; -import CardContent from '@material-ui/core/CardContent'; -import Button from '@material-ui/core/Button'; -import Grid from '@material-ui/core/Grid'; +import withStyles from "@material-ui/core/styles/withStyles"; +import { makeStyles, styled } from "@material-ui/core/styles"; +import { Link } from "react-router-dom"; +import Card from "@material-ui/core/Card"; +import CardMedia from "@material-ui/core/CardMedia"; +import CardContent from "@material-ui/core/CardContent"; +import Button from "@material-ui/core/Button"; +import Grid from "@material-ui/core/Grid"; -import Chip from '@material-ui/core/Chip'; -import Typography from '@material-ui/core/Typography'; -import AddCircle from '@material-ui/icons/AddCircle'; -import TextField from '@material-ui/core/TextField'; -import VerifiedIcon from '@material-ui/icons/CheckSharp'; -import Paper from '@material-ui/core/Paper'; -import GridList from '@material-ui/core/GridList'; -import GridListTile from '@material-ui/core/GridListTile'; -import GridListTileBar from '@material-ui/core/GridListTileBar'; -import Container from '@material-ui/core/Container'; +import Chip from "@material-ui/core/Chip"; +import Typography from "@material-ui/core/Typography"; +import AddCircle from "@material-ui/icons/AddCircle"; +import TextField from "@material-ui/core/TextField"; +import VerifiedIcon from "@material-ui/icons/CheckSharp"; +import Paper from "@material-ui/core/Paper"; +import GridList from "@material-ui/core/GridList"; +import GridListTile from "@material-ui/core/GridListTile"; +import GridListTileBar from "@material-ui/core/GridListTileBar"; +import Container from "@material-ui/core/Container"; // component -import '../App.css'; -import noImage from '../images/no-img.png'; -import Writing_Microblogs from '../Writing_Microblogs'; +import "../App.css"; +import noImage from "../images/no-img.png"; +import Writing_Microblogs from "../Writing_Microblogs"; const MyChip = styled(Chip)({ - margin: 2, - color: 'primary' + margin: 2, + color: "primary" }); const styles = { - button: { - positon: 'relative', - float: 'left', - marginLeft: 30, - marginTop: 20 - }, - paper: { - // marginLeft: "10%", - // marginRight: "10%" - }, - card: { - marginBottom: 10 - }, - profileImage: { - marginTop: 20 - }, - topicsContainer: { - border: 'lightgray solid 1px', - marginTop: 20, - paddingTop: 10, - paddingBottom: 10, - height: 300 - }, - addCircle: { - width: 65, - height: 65, - marginTop: 10 - }, - username: { - marginBottom: 100 - } + button: { + positon: "relative", + float: "left", + marginLeft: 30, + marginTop: 20 + }, + paper: { + // marginLeft: "10%", + // marginRight: "10%" + }, + card: { + marginBottom: 10 + }, + profileImage: { + marginTop: 20 + }, + topicsContainer: { + border: "lightgray solid 1px", + marginTop: 20, + paddingTop: 10, + paddingBottom: 10, + height: 300 + }, + addCircle: { + width: 65, + height: 65, + marginTop: 10 + }, + username: { + marginBottom: 100 + } }; class user extends Component { - state = { - profile: null, - imageUrl: null, - topics: null, - newTopic: null - }; + state = { + profile: null, + imageUrl: null, + topics: null, + newTopic: null + }; - handleDelete = (topic) => { - axios - .delete(`/deleteTopic/${topic.id}`) - .then(function() { - location.reload(); - }) - .catch(function(err) { - console.log(err); - }); - }; + handleDelete = topic => { + console.log(topic); + axios + .post(`/deleteTopic`, { + unfollow: topic + }) + .then(function() { + location.reload(); + }) + .catch(function(err) { + console.log(err); + }); + }; - handleAddCircle = () => { - axios - .post('/putTopic', { - topic: this.state.newTopic - }) - .then(function() { - location.reload(); - }) - .catch(function(err) { - console.log(err); - }); - }; + handleAddCircle = () => { + axios + .post("/putTopic", { + following: this.state.newTopic + }) + .then(function() { + location.reload(); + }) + .catch(function(err) { + console.log(err); + }); + }; - handleChange(event) { - this.setState({ - newTopic: event.target.value - }); - } + handleChange(event) { + this.setState({ + newTopic: event.target.value + }); + } - componentDidMount() { - axios - .get('/user') - .then((res) => { - this.setState({ - profile: res.data.credentials.handle, - imageUrl: res.data.credentials.imageUrl, - verified: res.data.credentials.verified ? res.data.credentials.verified : false - }); - }) - .catch((err) => console.log(err)); + componentDidMount() { + axios + .get("/user") + .then(res => { + this.setState({ + profile: res.data.credentials.handle, + imageUrl: res.data.credentials.imageUrl, + verified: res.data.credentials.verified + ? res.data.credentials.verified + : false, + topics: res.data.credentials.followedTopics + }); + }) + .catch(err => console.log(err)); - axios - .get('/getAllTopics') - .then((res) => { - this.setState({ - topics: res.data - }); - }) - .catch((err) => console.log(err)); + axios + .get("/getallPostsforUser") + .then(res => { + // console.log(res.data); + this.setState({ + posts: res.data + }); + }) + .catch(err => console.log(err)); + } - axios - .get('/getallPostsforUser') - .then((res) => { - console.log(res.data); - this.setState({ - posts: res.data - }); - }) - .catch((err) => console.log(err)); - } + render() { + const { classes } = this.props; + let authenticated = this.props.user.authenticated; - render() { - const { classes } = this.props; - let authenticated = this.props.user.authenticated; + let profileMarkup = this.state.profile ? ( +
loading username...
+ ); - let profileMarkup = this.state.profile ? ( -loading username...
- ); + let topicsMarkup = this.state.topics ? ( + this.state.topics.map( + topic => ( +loading topics...
+ ); - let topicsMarkup = this.state.topics ? ( - this.state.topics.map( - (topic) => ( -loading topics...
- ); + let imageMarkup = this.state.imageUrl ? ( +My Posts
+ ); - let postMarkup = this.state.posts ? ( - this.state.posts.map((post) => ( -My Posts
- ); - - // FIX: This needs to check if user's profile page being displayed - // is the same as the user who is logged in - // Can't check for that right now, because this page is always - // showing the logged in users profile, instead of retreiving the - // profile based on the URL entered - let editButtonMarkup = true ? ( + // FIX: This needs to check if user's profile page being displayed + // is the same as the user who is logged in + // Can't check for that right now, because this page is always + // showing the logged in users profile, instead of retreiving the + // profile based on the URL entered + let editButtonMarkup = true ? ( - + - ) : null; + ) : null; - return ( -