diff --git a/webmarker/src/componets/AddModal.js b/webmarker/src/componets/AddModal.js index 1eaf63b..ab036c9 100644 --- a/webmarker/src/componets/AddModal.js +++ b/webmarker/src/componets/AddModal.js @@ -52,6 +52,8 @@ const AddModal = ({onExit}) => { }); onExit(); + } else if(name != '' && link != '' && tag != '') { + alert('Please make sure the link is valid.\n\n(i.e. starts with "http"/"https")'); } else { diff --git a/webmarker/src/componets/List.js b/webmarker/src/componets/List.js index 1196c4a..dcbf150 100644 --- a/webmarker/src/componets/List.js +++ b/webmarker/src/componets/List.js @@ -1,10 +1,7 @@ import '../styles/List.css'; -import { useState } from 'react'; const List = ({data}) => { - const [reload, setReload] = useState(0); - - async function deleteEntity(id) { + function deleteEntity(id) { fetch("/delete", { // Adding method type diff --git a/webmarker_backend/server.js b/webmarker_backend/server.js index 25e56a3..f878fc8 100644 --- a/webmarker_backend/server.js +++ b/webmarker_backend/server.js @@ -22,16 +22,22 @@ app.post('/post', (req, res) => { let title; const pageToVisit = req.body.link; request(pageToVisit, (error, response, body) => { - if(response.statusCode === 200) { - // Parse the document body - const $ = cheerio.load(body); + try { + if(response.statusCode === 200) { + // Parse the document body + const $ = cheerio.load(body); - req.body.title = $('title').text(); - insertDoc(req.body); - } else { - req.body.title = null; - insertDoc(req.body); - } + req.body.title = $('title').text(); + insertDoc(req.body); + } else { + req.body.title = null; + insertDoc(req.body); + } + } catch (error) { + console.log(error); + req.body.title = null; + insertDoc(req.body); + } }); });