CRClothier / My-Portfolio-Website

A portfolio website built using HTML, CSS & Javascript, showing all the projects I have created to date and acting as a CV.

MIT License

11 stars 0 forks source link

Bugs: hide and seek #13

Open MacCrazyman opened 2 years ago

MacCrazyman commented 2 years ago

Debugging portfolio project

Hi @CRClothier, I was checking the errors inside the Debugginng branch of your portfolio, those are the issues I found so far:

[ ] There is a problem in line 64 of your JavaScript file with the class assignment. https://github.com/CRClothier/My-Portfolio-Website/blob/be6fe2c374210adcdcc8bc63f157aa4123412e28/script.js#L64 this problem can ve solved using the proper indexOf method
```
 card.className = `project item${projects.indexOf(project)}`;
```

[ ] The other problem I could notice so far is related to the JSON API. https://github.com/CRClothier/My-Portfolio-Website/blob/be6fe2c374210adcdcc8bc63f157aa4123412e28/script.js#L176 Instead of the parse method, you should use stringify to store the data in a proper way.
```
const saveData = JSON.stringify(formData);
```

Hope this information to be useful. Keep the hard work going. 👍