Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugs: JavaScript #33

Open
2 tasks done
GraceAbike opened this issue Jul 29, 2022 · 1 comment
Open
2 tasks done

Bugs: JavaScript #33

GraceAbike opened this issue Jul 29, 2022 · 1 comment

Comments

@GraceAbike
Copy link
Collaborator

JavaScript Bugs Found In Your Project:

Summary

After conducting a thorough debugging process in your Portfolio Project I found 2 bugs.

Here is the list of the found bugs in your JavaScript files:

  • In your assets/js/index.js file line number 111 you assigned projectsData[pid].name to h2.innerText variable without converting it to a string beforehand. Thus, some unexpected behaviors may occur. To fix this bug, you must convert the projectsData[pid].name.value.toString() to a string in the first place.
  • In your assets/js/index.js file line number 213, you called the createProjectCard(projId-1) and passed the projId-1 as a parameter without converting the projId to an Integer. Thus, some unexpected behaviors may occur. To fix this bug, you must convert the projId before passing it to the function using the JavaScript built-in function parseInt().
@ZikaZaki
Copy link
Owner

Hi, @GraceAbike. Thanks for your thorough debugging on my Portfolio Project. Regarding to the bug list you have commented, I will make sure to solve them following your suggestions. Thanks again for your time and cooperation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants