README.md

FACBOOK

http://facboook.herokuapp.com/

• we created skeleton file structure together first - setting up express and evironment variables
• we created a kanban, without giving estimate timings
• at this point, we didn't discuss yet what our project would be
• we agreed to decide what our app would be at the point of needing to create our schema (but before creating them!)

Design

• Used Figma to produce the following:

Code Review / Learnings:

Multiple queries

• trying to insert into different tables in one query.

Oli's workshop with one table using an Array (called values) created by passing requeast.boddy in to Object.values():

function post(request, response) {
  const data = request.body;
  const values = Object.values(data);
  db.query(
    "INSERT INTO users(username, age, location) VALUES($1, $2, $3)",
    values
  ).then(() => {
    response.redirect("/");
  });
}

Our effort 1: 1 DB query (as per oli's) but with 2 SQL lines(1 for each table); Array of values created the same way as Oli's

function post(request, response) {
  const data = request.body;
  const values = Object.values(data);
  db.query(
    `INSERT INTO people(name, github_username, pronoun, cohort, location) VALUES($1, $2, $3, $5, $6)
    INSERT INTO interests(username, activity) VALUES($2, $4);`,
    values
  )
    .then(() => {
      response.redirect("/");
    });
}

UnhandledPromiseRejectionWarning: error: syntax error at or near "INSERT"

Our effort 2: A second DB query (1 for each table); ; Array of values created the same way as Oli's

function post(request, response) {
  const data = request.body;
  const values = Object.values(data);
  db.query(
    `INSERT INTO people(name, github_username, pronoun, cohort, location) VALUES($1, $2, $3, $5, $6);`,
    values
  )
    .then(() => {
      db.query(
        "INSERT INTO interests(username, activity) VALUES($2, $4); ",
        values
      );
    })
    .then(() => {
      response.redirect("/");
    });
}

UnhandledPromiseRejectionWarning: error: could not determine data type of parameter $4

• Our effort 3: Rather than using array created by new Object.values(data), we created two arrays containing the data that we wanted (give exmaple) [data.name, data.cohort]

function post(request, response) {
  const data = request.body;
  const justPeopleValues = [data.name, data.github_username, data.pronoun, data.cohort, data.location,
  ];
  const hobbyValues = [data.github_username, data.interest];
  db.query(
    `INSERT INTO people(name, github_username, pronoun, cohort, location) VALUES($1, $2, $3, $4, $5);`,
    justPeopleValues
  )
    .then(() => {
      db.query(
        "INSERT INTO interests(username, activity) VALUES($1, $2); ",
        hobbyValues
      );
    })
    .then(() => {
      response.redirect("/");
    });
}

Cypress testing

Error message: relation 'interests' does not exists - on before each tests, to reset the db - we realised that our table name had been updated but we hadn't udated our DROP TABLE IF EXISTS command;

SSL

When we first tried to delpoy on Heroku we kept getting an error message and not being able to access the page. We eventually worked out that our error was here:

UnhandledPromiseRejectionWarning: error: no pg_hba.conf entry for host "3.238.31.155", user "glikkvufsehtps", database "d7uv40jch5kjln", SSL off

We basically overrided this:

  ssl: {
        rejectUnauthorized: false
      }
};

Lessons Learnt (SCG)

Stop:

Continue:

• Co-authored-by: Rosie O'Donnell [email protected]

Go:

• Stop using add .
• Improve on Commit messages