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

Issue 1938 #1994

Merged
merged 16 commits into from
May 12, 2021
Merged

Issue 1938 #1994

merged 16 commits into from
May 12, 2021

Conversation

ZoeLeBlanc
Copy link
Member

@ZoeLeBlanc ZoeLeBlanc commented Jan 25, 2021

This PR is a first attempt at closing #1938. It adds two fields to lessons.html and snippets.yaml, a tested_date and lesson-tester so that we can begin to record testing of lessons.

The names of lesson testers will appear next to reviewers, editors, translators, etc... whereas the lesson tested date will appear below next to published, modified, translated date, etc...

This setup assumes that we will only record the most recent tested date, but that we will record all previous lesson testers. The rationale is that though we could include all testing dates that info is less relevant (we mostly want to know if it's stale or not) and we want to give credit to all testers. So therefore list of names, but only one date to try and minimize user entered data errors.

Prior to merging this PR, we'll need to update lesson guidelines. Also open to discussing additional fields or formats!

Checklist

If 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.

@acrymble
Copy link

Just in terms of a timeline @ZoeLeBlanc, I'm hoping to have a placement student to test some lessons in May. Are we likely to have this live in time, or will we have to keep track on a spreadsheet or something else in the meantime?

@ZoeLeBlanc
Copy link
Member Author

We should have it available pretty soon @acrymble. I think the technical team is about done on our end so going to make this PR live and request translation from various teams!

@ZoeLeBlanc ZoeLeBlanc marked this pull request as ready for review March 24, 2021 11:56
@ZoeLeBlanc
Copy link
Member Author

Would members of @programminghistorian/spanish-team @programminghistorian/french-team @programminghistorian/portuguese-team be able to translate some snippets for this PR?

I'm happy to add in the data or have people merge into this branch, but I need translations of the words tested by and either last tested or tested on (which ever makes more sense to you) for this PR.

You can see the snippets fields below:

lesson-testers:
  en: tested by
  es:
  fr: 
  pt: 
tested-date:
  en: tested
  es:
  fr: 
  pt: 

Thanks!

@pymentha
Copy link
Contributor

pymentha commented Mar 24, 2021 via email

@ZoeLeBlanc
Copy link
Member Author

Thanks so much @pymentha 🙏🏽 ! Just merged in your translations 🎉

@rivaquiroga
Copy link
Member

@programminghistorian/spanish-team: Which translation do you think works best in this case? testeado por y último test, probado por y última prueba, ¿otra?

@ZoeLeBlanc
Copy link
Member Author

@programminghistorian/spanish-team would anyone have time to add a translation to this branch so we could merge this in? Thanks 🙏🏽 !

@rivaquiroga
Copy link
Member

@ZoeLeBlanc, done!

@ZoeLeBlanc
Copy link
Member Author

Thanks so much @rivaquiroga ! Just one final issue for the @programminghistorian/spanish-team to chime in on.

There's a breaking link in the lesson introduccion-control-versiones-github-desktop. It looks like the link to the old Github Desktop is no longer available (https://central.github.com/mac/latest ). When I went to the web archive to find an alternative link, it redirected to a zip download https://mac-installer.github.com/mac/GitHub%20Desktop%20222.zip but I have no idea if that's suitable for the lesson or not. Would someone mind letting me know and I can update the broken link. Thanks again!!

@jenniferisasi
Copy link
Contributor

jenniferisasi commented May 11, 2021

Hi @ZoeLeBlanc. That lesson is retired, so no need to maintain it, but also makes me wonder (cos I don't remember) if it is necessary that the build runs inside the retired lessons to look for broken links?
https://programminghistorian.org/en/lesson-retirement-policy

@jenniferisasi jenniferisasi mentioned this pull request May 11, 2021
7 tasks
@jenniferisasi jenniferisasi merged commit 7e32e39 into gh-pages May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants