-
Notifications
You must be signed in to change notification settings - Fork 0
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
Comments
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
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:
assets/js/index.js
file line number111
you assignedprojectsData[pid].name
toh2.innerText
variable without converting it to astring
beforehand. Thus, some unexpected behaviors may occur. To fix this bug, you must convert theprojectsData[pid].name.value.toString()
to a string in the first place.assets/js/index.js
file line number213
, you called thecreateProjectCard(projId-1)
and passed theprojId-1
as a parameter without converting theprojId
to anInteger
. Thus, some unexpected behaviors may occur. To fix this bug, you must convert theprojId
before passing it to the function using the JavaScript built-in functionparseInt()
.The text was updated successfully, but these errors were encountered: