Skip to content

Import catalogus data into collection #141

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

Open
lukavdplas opened this issue May 11, 2023 · 1 comment
Open

Import catalogus data into collection #141

lukavdplas opened this issue May 11, 2023 · 1 comment
Assignees

Comments

@lukavdplas
Copy link
Contributor

Users should be able to import results from a catalogus (CentreForDigitalHumanities/edpop-explorer#13) into a collection.

They can do this in two ways:

  • Manually select records -> the frontend sends a list of IDs
  • Select everything from a query (which may not all be loaded into the frontend) -> the frontend sends the query

This task is best handled asynchronously, i.e. through celery.

@lukavdplas lukavdplas added this to EDPOP Feb 8, 2024
@lukavdplas lukavdplas moved this to Todo in EDPOP Feb 8, 2024
@lukavdplas lukavdplas self-assigned this Feb 8, 2024
@jgonggrijp
Copy link
Contributor

@lukavdplas via Teams:

ok, ik heb even gekeken en het laatste wat ik rondom de records in een collectie had gedaan stond in #195. Daar heb ik nu nog wat dingen afgewerkt, dus ik denk dat die gemerged kan worden. Dan is de data / api dus ook klaar om te worden uitgebreid met records toevoegen

@lukavdplas lukavdplas assigned jgonggrijp and unassigned lukavdplas Dec 11, 2024
@jgonggrijp jgonggrijp moved this from Todo to In Progress in EDPOP Dec 12, 2024
jgonggrijp added a commit that referenced this issue Jan 13, 2025
This appears to have been an omission from #181.
jgonggrijp added a commit that referenced this issue Jan 13, 2025
…142)

Using the Postgres RDBMS for this as it is probably cleaner (and more
convenient) to keep this out of the triplestore.
jgonggrijp added a commit that referenced this issue Feb 14, 2025
…on (#141 #142)"

At second thought, I think I should store this information in the
triplestore. I discussed this with @tijmenbaarda and he agrees.

This reverts commit 0c9ba80.
jgonggrijp added a commit that referenced this issue Mar 12, 2025
If marked with @AbstractMethod, it is impossible to derive classes
that don't define these methods. The alternative, exception-raising
approach taken here ensures that the methods can be omitted if set and
clear are defined instead.
jgonggrijp added a commit that referenced this issue Mar 17, 2025
This appears to have been an omission from #181.
jgonggrijp added a commit that referenced this issue Mar 17, 2025
…142)

Using the Postgres RDBMS for this as it is probably cleaner (and more
convenient) to keep this out of the triplestore.
jgonggrijp added a commit that referenced this issue Mar 20, 2025
jgonggrijp added a commit that referenced this issue Mar 20, 2025
This appears to have been an omission from #181.
jgonggrijp added a commit that referenced this issue Mar 20, 2025
…142)

Using the Postgres RDBMS for this as it is probably cleaner (and more
convenient) to keep this out of the triplestore.
jgonggrijp added a commit that referenced this issue Mar 20, 2025
…on (#141 #142)"

At second thought, I think I should store this information in the
triplestore. I discussed this with @tijmenbaarda and he agrees.

This reverts commit 0c9ba80.
jgonggrijp added a commit that referenced this issue Mar 20, 2025
If marked with @AbstractMethod, it is impossible to derive classes
that don't define these methods. The alternative, exception-raising
approach taken here ensures that the methods can be omitted if set and
clear are defined instead.
jgonggrijp added a commit that referenced this issue Mar 20, 2025
jgonggrijp added a commit that referenced this issue Mar 20, 2025
…142)

This restores the ability to add records to collections and also
improves the separation of concerns. The VRECollectionView is
responsible for selecting collections and for detecting when the user
wants to submit. Its parent views are responsible for passing the URIs
of the models that should be added to those collections.
jgonggrijp added a commit that referenced this issue Mar 20, 2025
This supersedes the nestSubject function, which handled one subject at
a time. The enforest function can be more efficient because it caches
the previously obtained results for other subjects in the graph, and
it has the option to automatically detect top-level subjects.

Cf #141 #195 #242 #183
jgonggrijp added a commit that referenced this issue Mar 20, 2025
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