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

Update information for Marvin Beckers #774

Closed
wants to merge 1 commit into from

Conversation

embik
Copy link

@embik embik commented Dec 20, 2024

Hey folks! 👋🏻

I would like to add some more information to my profile, so here is a PR that updates things and puts them a bit more into standard (I think the personal website in the bio is kind of unusual, and there's a field for it anyway).

I'm a little confused about slack vs slack_id. Many entries have the slack field, but slack_id seems to be the right one according to schema.json. If this correction is not accurate, please let me know and I will drop it from the commit.

@cjyabraham
Copy link
Contributor

Thanks for calling this out. I've replaced all "slack" attributes with the correct "slack_id". This caused conflicts in this PR, so I made the changes you wanted manually. If I made an error or you want something else changed, please open a new PR.

@cjyabraham cjyabraham closed this Dec 20, 2024
@embik embik deleted the update-marvin-beckers branch December 20, 2024 20:12
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.

2 participants