Skip to content

Update module for DKAN 4.x compatibility - #7

Open
dcgoodwin2112 wants to merge 2 commits into
mainfrom
feature/dkan-4-compatibility
Open

Update module for DKAN 4.x compatibility#7
dcgoodwin2112 wants to merge 2 commits into
mainfrom
feature/dkan-4-compatibility

Conversation

@dcgoodwin2112

@dcgoodwin2112 dcgoodwin2112 commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

Updates Datastore Data Preview for DKAN 4.x compatibility.

  • Updates Drupal/DKAN module dependencies to DKAN 4 namespaces (dkan_common, dkan_datastore, dkan_metastore)
  • Updates datastore/common/metastore PHP namespaces used by services, controllers, and tests
  • Adds DKAN 4 metastore response handling via RootedJsonData normalization
  • Keeps support for resolving distribution UUIDs to {identifier}__{version} datastore resource IDs
  • Updates README, agent instructions, and lightweight architecture docs for DKAN 4.x behavior

Testing

  • Run unit tests
  • Enable the module on a DKAN 4.x site
  • Confirm a dataset node with an importable/tabular distribution renders a data preview table.
  • Confirm the dataset edit form shows a “Preview data” link beside tabular distribution  downloadURL  fields.
  • Confirm sort, pager, and page-size interactions update the preview table without a full page reload.

Co-authored-by: Copilot

dcgoodwin2112 and others added 2 commits July 2, 2026 13:22
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dcgoodwin2112
dcgoodwin2112 marked this pull request as ready for review July 2, 2026 18:03
@dcgoodwin2112 dcgoodwin2112 changed the title Update module for DKAN 4 compatibility Update module for DKAN 4.x compatibility Jul 2, 2026
Comment thread composer.json
},
"require-dev": {
"drupal/core": "^10.2 || ^11",
"getdkan/rooted-json-data": "^1.0",

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

We added getdkan/rooted-json-data as a dev dependency because DKAN 4’s MetastoreService::get() is typed to return RootedData\RootedJsonData, and the module’s standalone PHPUnit setup needs that class available when mocking/testing MetastoreService.

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