-
-
Notifications
You must be signed in to change notification settings - Fork 881
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.
Pls update with requested changes
data.json
Outdated
@@ -1217,4 +1217,10 @@ | |||
"imageUrl": "https://user-images.githubusercontent.com/54446880/127509715-b594b68d-4703-4bc2-aed5-1172d7df6a5c.png", | |||
"issueNumber": 627 | |||
} |
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.
As @deephunt3r mentioned here is a comma missing
Co-authored-by: Rakesh <60481830+deephunt3r@users.noreply.github.com>
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.
LGTM, I think this can be merged now @schmelto
Sounds good ✌🏼 |
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.
LGTM 👍🏻 Thanks for the contribution 🎊
Just realised that there is conflicts that need resolving |
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.
LGTM.
great ✨ |
Co-authored-by: Tom Schmelzer <30869493+schmelto@users.noreply.github.com>
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.
lgtm
Thanks for your contribution 🚀 |
resolve #631