-
-
Notifications
You must be signed in to change notification settings - Fork 87
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
Automatic schedule & artifact update #918
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for ols-bebatut ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
91a355a
to
174fc9c
Compare
@@ -595,7 +595,7 @@ weeks: | |||
- slides: | |||
https://drive.google.com/file/d/1w8AUnsbwSpqfEK0YxKmSZGIeIlEbppSN/view?usp=share_link | |||
speakers: | |||
- gemmaturon |
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.
@NPDebs, could you look at why Gemma and a few others are not found anymore? Thanks a lot!
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.
Will do. 👍🏽
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 a lot!!!!
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.
Okay, so far I have found something that's causing Gemma
and Fadanka
to be missing.
Gemma exists in L4338 and Fadanka in L3993 of the people.yaml
file.
However, both individuals had their names updated in PR #907 to include their additional names:
- Gemma Turon Rodrigo (see diff) and
- Wapouo Fadanka Stephane (see diff).
It no longer recognises this Gemma Turon as Gemma Turon Rodrigo.
Solutions:
- Update the Google Sheets or other data sources to match the new names.
- Use partial matching or regular expressions: This can become problematic if we have very similar names (e.g Umar Farouk Ahmad and Umar Ahmad).
Let me know what you think. 🙏
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.
The first solution is probably the best one. It is the one I used before
174fc9c
to
801c713
Compare
Automatic schedule update and artifacts done via GitHub Action once a week