-
Notifications
You must be signed in to change notification settings - Fork 229
Josir Gomes Bio #1839
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
Josir Gomes Bio #1839
Conversation
|
Hi @josircg! Welcome to the team. I added the Spanish translation of your Bio + the missing team roles. |
|
Thank you @rivaquiroga . I will add the affiliation and photo on a second commit. I was in doubt if I have to have travis installed on my machine to send the pull request so I did a small change to test. I could not assign myself to the issue and I thought this could be a problem either. |
|
@josircg - Travis is a web service that doesn't need to be installed on your computer. And you should be able to assign yourself to a pull request, but ultimately that shouldn't cause any errors if some quirk of the interface won't let you do so. It's more about keeping things organized and tidy. |
|
@josircg, maybe you can't assign yourself because you are not yet part of the team here in Github. Did you inform Zoe about your github username? |
|
No. I imagined that it was informed already. But I did it now. |
|
Josir already mentioned his username in this message (#1830 (comment)). Probably the problem was the opening of another issue. |
|
@josircg can you please use the checklist in the original comment above? This helps make sure you've done all the checks before someone reviews it. It's a series of reminders of how to do a proper pull request. |
Hi. I don't have access yet to do any change on the original issues (#1830), or assign myself to them. I've already asked for help to @ZoeLeBlanc |
|
Well... I just found the invitation on my spam box. And it's expired :/ |
|
@rivaquiroga I think everything is ok now. Need only your final approval. |
|
@josircg, the french translation is missing. You need to tag the french team + Marie Christine to ask for the translation. |
I asked someone else to translate but I'm tagging @mariechristineb to verify if it's ok. |
mariechristineb
left a comment
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.
I didn't want to start a new branch so I added my changes to the translation in a comment
|
Did you include your photo in the pull request @josircg ? |
No. I will do it in a second commit because I have to find/assemble a squared black/white photo and didn't want to late the onboarding process. Photo added now. |
|
@josircg, it seems that the only thing missing is your affiliation. |
|
Last thing and then we are ready to merge: the file name of your photo must be the same you use in |
|
Hi @rivaquiroga. Should I have to close the issue or is it closed only when deployed? |
|
Hi @josircg We're are just waiting for the review from @mariechristineb and then we can close the ticket. Is this correct @rivaquiroga ? |
Closes #1838
Checklist
Closes #ISSUENUMBERto the description aboveIf you are having difficulty fixing Travis errors, first consult https://github.com/programminghistorian/jekyll/wiki/Making-Technical-Contributions carefully, especially "Common Travis Errors". Then contact the technical team if you need further help.