Time tracking dashboard

Community feedback

• 

  Shuaib•640

  @JustShuaib

  Posted about 3 years ago

  Hi there! I just wanted to point out a few things. While fetching the data from the json file, always include a way to catch errors incase the request fails to be successful.

  You had

  document.addEventListener("DOMContentLoaded", () => {
  // function async to get data from the json file
  async function loadObjJson() {
  const response = await fetch("./assets/js/data.json");
  const data = await response.json();
  console.log(data); 
  

  Declare a function in a separate block then call the function inside the DOMContentLoaded event listener,instead of including the function block inside the event listener. Also, always remove console logs from your code when you're done

  Also for your titles, instead of selecting them individually,you could give all the headings the same class and querySelectorAll all the headings then loop through the data to display each in the respective title.

  Happy coding!✌️😊

  13

  Marked as helpful