Problem/Motivation

The current method has an endpoint that just gathers all the currently published datasets for a given theme and zips them up. This is not very efficient. They might be cached fine but probably aren't. We might need a way to delineate one current from another, like a hash of all the modified dates strung together.

This may not be needed, but creating an issue so it does not slip through the cracks.

Acceptance Criteria

Testing steps

Issue fork dkan_dataset_archiver-3544036

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

swirt created an issue. See original summary.

swirt’s picture

swirt
he/him
Primary language English
commented

This is needed for the current functionality on PDC.
Solution is something like this:

  1. After as part of the theme aggregation the theme name is kept and used to trigger a current aggregation.
  2. It will trigger its own cache tag invalidation
  3. It will load all published datasets and add them to a zip.
  4. Of course there will have to be separate aggregation of private.
  5. Manifest should also be included.
swirt’s picture

swirt
he/him
Primary language English
commented

Needs a setting too or enable current aggregation.

swirt’s picture

swirt
he/him
Primary language English
commented
Assigned: Unassigned » swirt

swirt opened merge request !40

  • swirt committed 479896a6 on 1.0.x 
    [#3544036] feat: Replace method for downloading current snapshot
swirt’s picture

swirt
he/him
Primary language English
commented

This has been completed. The "current" api endpoint now exists.

swirt’s picture

swirt
he/him
Primary language English
commented
Status: Active » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

swirt’s picture

swirt
he/him
Primary language English
commented
Status: Fixed » Closed (fixed)

Released as part of 1.0.0-alpha15