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

(phi0690)-phi0690.phi001-phi0690.phi003.perseus-eng2/lat2_cts.xml #432

Merged
merged 2 commits into from
Jun 4, 2021

Conversation

AlisonBabeu
Copy link
Contributor

@AlisonBabeu AlisonBabeu commented Jun 3, 2021

Corrected _cts.xml for phi0690.phi003.perseus-eng2 also updated the file header to current standards.
Updated headers for both phi0690.phi003.perseus-lat2 and perseus-eng2

Fix #430

Per issue #431.

Corrected _cts.xml for phi0690.phi003.perseus-eng2 also updated the file header to current standards.
Updated headers for both phi0690.phi003.perseus-lat2 and perseus-eng2
@AlisonBabeu AlisonBabeu requested a review from lcerrato June 3, 2021 17:17
@ghost
Copy link

ghost commented Jun 3, 2021

Hook UI build recap

@@ -36,18 +35,15 @@
<biblStruct>
<monogr>
<author>Vergil</author>
<author>Virgil</author>
<title>The Bucolics, Aeneid, and Georgics Of Virgil</title>
<editor role="editor">J. B. Greenough</editor>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<editor role="editor"> s/b <editor>

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not quite sure what you mean by s/b @lcerrato sorry if I am missing the obvious.

Copy link
Contributor

@lcerrato lcerrato Jun 3, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, “should be”

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So its ok that I took out the role="editor" right that's what I have been doing.

<respStmt>
<name>Theodore C. Williams</name>
<resp>trans.</resp>
<resp>translator</resp>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably preferable practice to use <editor role="translator"> given what Eldarion is doing with <respStmt> tags

phi0690.phi001-phi0690.phi003

Corrected metadata and cts for phi0690.phi001.perseus-eng2 and phi0690.phi002.perseus-eng2.
Standardized CTS descriptions and TEI-headers.
@AlisonBabeu AlisonBabeu changed the title (phi0690)-phi0690.phi003.perseus-eng2/lat2 (phi0690)-phi0690.phi001-phi0690.phi003.perseus-eng2/lat2_cts.xml Jun 4, 2021
@ghost
Copy link

ghost commented Jun 4, 2021

Hook UI build recap

@AlisonBabeu AlisonBabeu merged commit f912a0f into master Jun 4, 2021
@AlisonBabeu AlisonBabeu deleted the phi0690 branch June 4, 2021 17:41
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.

Misattribution of translation - Vergil's Aeneid
2 participants