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

Restructure database file hierarchy #240

Merged
merged 4 commits into from
Feb 17, 2024
Merged

Restructure database file hierarchy #240

merged 4 commits into from
Feb 17, 2024

Conversation

cmil
Copy link
Member

@cmil cmil commented Feb 6, 2024

The new structure puts all files (tei, rdf, metrics, sitelinks) for an individual play into a single collection instead of spreading them over different type specific collections with sub paths for each corpus. This makes it much easier to delete a single play or an entire corpus since we no longer need to do housekeeping in several collections with a delete trigger.

The new structure looks similar to this:

    db
    |- dracor
      |- corpora
        |- ger
        | |- lessing-emilia-galotti
        | | |- tei.xml
        | | |- metrics.xml
        | | |- rdf.xml
        | | |- sitelinks.xml
        | |- zschokke-abellino
        |   |- ...
        |- als
        |- ita
        |- ...

cmil added 3 commits February 5, 2024 09:45
The new structure puts all files (tei, rdf, metrics, sitelinks) for
an individual play into a single collection instead of spreading them
over different type specific collections with sub paths for each
corpus. This makes it much easier to delete a single play or an
entire corpus since we no longer need to do housekeeping in several
collections with a delete trigger.

db
|- dracor
  |- corpora
    |- ger
    | |- lessing-emilia-galotti
    | | |- tei.xml
    | | |- metrics.xml
    | | |- rdf.xml
    | | |- sitelinks.xml
    | |- zschokke-abellino
    |   |- ...
    |- als
    |- ita
    |- ...
The function now removes the entire corpus collection instead of iterating
over each TEI file.

Solves #221
@cmil cmil force-pushed the restructure-file-hierarchy branch from ac88e9c to 44ac7b1 Compare February 9, 2024 18:39
@cmil cmil force-pushed the restructure-file-hierarchy branch from 44ac7b1 to f3c720f Compare February 17, 2024 19:11
@cmil cmil self-assigned this Feb 17, 2024
@cmil cmil merged commit e5ef9e5 into main Feb 17, 2024
1 check passed
@cmil cmil deleted the restructure-file-hierarchy branch February 17, 2024 19:21
cmil added a commit to dracor-org/testdracor that referenced this pull request Feb 17, 2024
Testing removal with webhook after refactoring of dracor-org/dracor-api#240.
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.

1 participant