-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
data: Prashanth-bokkala #10030
data: Prashanth-bokkala #10030
Conversation
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.
Please make the necessary changes to get your profile merged.
yes , but they shown in the tutorial about this so i included that , and mainly i checked the code with Playground in that , its shown that its a valid code , i checked twice before made pull request , if you still need changes please let me know |
Yes those fields were required but now they are not required anymore so there is no need to add it. The JSON playground only checks for whether the json is valid or not. |
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 👍
Welcome to Eddiehub community ✨✨
Fixes Issue
Changes proposed
Check List (Check all the applicable boxes)
Screenshots
Note to reviewers