Skip to content
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 Jeremy Fiel as an ambassador #781

Merged

Conversation

jeremyfiel
Copy link
Contributor

Please let me know if I need anything further.

thanks
Jeremy

@jeremyfiel jeremyfiel force-pushed the feat/add-jeremy-fiel-ambassador branch from 9eeb861 to 5d14a39 Compare August 2, 2024 19:35
@jeremyfiel jeremyfiel force-pushed the feat/add-jeremy-fiel-ambassador branch from 5d14a39 to d6f0be9 Compare August 2, 2024 19:45
Copy link
Member

@Julian Julian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are changes here to other parts of the file other than the new section / the diff is large, which likely is the reason there's a merge conflict.

After backing those out though this has my support personally.

Copy link
Collaborator

@benjagm benjagm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Huge thanks for your interest in becoming an Ambassador.
Everything looks great to me.

@benjagm
Copy link
Collaborator

benjagm commented Aug 22, 2024

there's a merge conflict.

I just fixed it to make things easier.

Comment on lines +312 to +329
{
"type": "other",
"title": "JSON Schema Slack Community Member",
"date": {
"year": 2021,
"month": "November"
},
"link": "https://json-schema.slack.com"
},
{
"type": "other",
"title": "JSON Schema Stack Overflow Community Member",
"date": {
"year": 2021,
"month": "November"
},
"link": "https://stackoverflow.com/questions/tagged/json-schema"
},
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we need a "community presence" field for these. This seems common enough to warrant its own field.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we need a "community presence" field for these. This seems common enough to warrant its own field.

I just created a PR to add a new value in the enum referring to Community contributions:

#791

@jeremyfiel
Copy link
Contributor Author

There are changes here to other parts of the file other than the new section / the diff is large, which likely is the reason there's a merge conflict.

After backing those out though this has my support personally.

Some of the prior entries violated the enums and the formatting was incorrect

@benjagm benjagm merged commit e4c8884 into json-schema-org:main Aug 24, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants