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

As a content admin, I want TEI contributors documented in the new GitHub annotation and transcription backups so that there is a record of everyone who has contributed to the transcription structure and content. #1093

Closed
3 tasks done
rlskoeser opened this issue Sep 19, 2022 · 1 comment
Assignees

Comments

@rlskoeser
Copy link
Contributor

rlskoeser commented Sep 19, 2022

testing notes

Use this branch of our test transcription backup to test:
https://github.com/Princeton-CDH/test-pgp-annotations/tree/test-tei-migration

Maybe easiest to look at the list of recent commits

  • there are transcription export files in this branch of the repository created by the migration (this branch was empty until I ran the migration script)
  • commit message includes TEI filename
  • commit message / credit is based on who contributed to the bitbucket file (click in to see "co-authored-by" text in the full commit message; I only have a github co-author email for myself right now, so only my commits are linked to my github account)

FYI: got a weird error and the tei to annotation script didn't finish running; I will need to re-run it and investigate, but I think it got far enough for you to test that it's correctly exporting to github at migration time and preserving contributor information.

dev notes

Revise tei to annotation migration script to add a backup step. As each document is converted, determine the contributors, generate a co-author list, and run the github export. Should commit but not push until all content has been migrated.

@rlskoeser rlskoeser added this to the CDH/PGP end of grant year 2 milestone Sep 19, 2022
@rlskoeser rlskoeser self-assigned this Sep 22, 2022
@rlskoeser rlskoeser added the 🗜️ awaiting testing Implemented and ready to be tested label Sep 23, 2022
@kseniaryzhova
Copy link

@rlskoeser works for me!

@rlskoeser rlskoeser removed the 🗜️ awaiting testing Implemented and ready to be tested label Sep 26, 2022
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

No branches or pull requests

2 participants