-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
data: sandramsc #6113
data: sandramsc #6113
Conversation
updated structure & added new milestone
fix json syntax error at "Web3"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's great having you contribute to this project
Welcome to the community 🤓If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.
json syntax fix -#3
not sure why the JSON check keeps failing: I restructured and edited the json multiple times to debug where this error is, even by copying and pasting Eddies's json into mine (passed all checks) -- then replacing it with my data which failed 3 more times... |
data/sandramsc.json
Outdated
} | ||
"icon": "FaLinkedin", | ||
"color": "#0000FF" | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove comma
}, | |
} |
I copied/pasted the file into jsonlint.com and hopefully the inline comment I have made resolves this. We should be able to merge your PR once you change the information back to your own name/bio |
JSON syntax fix
Thanks @SaraJaoude! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good
fix json syntax error at "Web3"