Skip to content

feat: add data dictionary table snippet#121

Merged
jguo144 merged 3 commits into
masterfrom
jguo144/2026-05-14/fix-data-dictionary-display
May 15, 2026
Merged

feat: add data dictionary table snippet#121
jguo144 merged 3 commits into
masterfrom
jguo144/2026-05-14/fix-data-dictionary-display

Conversation

@jguo144
Copy link
Copy Markdown
Collaborator

@jguo144 jguo144 commented May 14, 2026

Description

For datastore-backed resources that have an active data dictionary, the resource read page renders the data dictionary (column, type, label, description) above “Additional Information”. The existing dictionary download control remains when a dictionary is present. This layout keeps the dictionary UI in the theme so the datastore plugin can be listed after this extension in ckan.plugins without losing the dictionary block on the resource page.

Changes

  • Add datastore/snippets/dictionary_view.html with a striped table and “toggle more” behavior after ten rows
  • Extend package/resource_read.html to load the dictionary via h.datastore_dictionary, gate on h.opendata_theme_is_data_dict_active and include the snippet next to the download button.

Copy link
Copy Markdown

@orca-security-us orca-security-us Bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@jguo144 jguo144 requested a review from nolanpruett May 15, 2026 15:54
@jguo144 jguo144 enabled auto-merge (squash) May 15, 2026 15:58
@jguo144 jguo144 merged commit 6fca57a into master May 15, 2026
5 checks passed
@jguo144 jguo144 deleted the jguo144/2026-05-14/fix-data-dictionary-display branch May 15, 2026 17:19
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.

2 participants