-
-
Notifications
You must be signed in to change notification settings - Fork 41
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
Add Juan Cruz Viotti to ambassador list #761
Conversation
Yes, they very much count =] |
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.
So pleased to formally recognise the work you do and the effort you've put into supporting JSON Schema!
Thanks a lot! ❤️
@Relequestual Should I add them to this PR? |
I added them in case you want to re-review |
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.
Thanks Juan!!
Hi @jviotti . Can you please resolve the conflict? We'll merge this asap. |
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
1a8067b
to
8e42cac
Compare
Done @benjagm ! |
Looks like the original JSON, even before this PR, is invalid? Let me fix it here |
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
OK @benjagm I also found the previous ambassador entry was not valid according to the schema you had in there. It should be all fixed now |
Not sure if I'm missing anything, but please let me know. Does things like AlterSchema and Learn JSON Schema count?