Skip to content

Add config option to hide resource information#21

Merged
jguo144 merged 2 commits into
masterfrom
jguo144/2026-04-06/link-config
Apr 6, 2026
Merged

Add config option to hide resource information#21
jguo144 merged 2 commits into
masterfrom
jguo144/2026-04-06/link-config

Conversation

@jguo144
Copy link
Copy Markdown
Collaborator

@jguo144 jguo144 commented Apr 6, 2026

Description

  • Allow hiding the resource information link (package/resource name) displayed at the bottom of data tables
  • Add configurable hide_resource_info option to both og_datatables_view and og_datatables_datefilter_view
  • Defaults to False to maintain backward compatibility
  • Can be configured per view or globally via ckan.datatables.hide_resource_info_default config
  • Adjusted CSS pagination margins when resource info is hidden

@jguo144 jguo144 changed the title Jguo144/2026 04 06/link config Add config option to hide resource information Apr 6, 2026
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 enabled auto-merge (squash) April 6, 2026 18:32
@jguo144 jguo144 requested a review from surajdeval April 6, 2026 18:33
@jguo144 jguo144 requested a review from nolanpruett April 6, 2026 18:33
Copy link
Copy Markdown

@surajdeval surajdeval left a comment

Choose a reason for hiding this comment

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

LGTM

@jguo144 jguo144 merged commit 70d3cca into master Apr 6, 2026
9 checks passed
@jguo144 jguo144 deleted the jguo144/2026-04-06/link-config branch April 6, 2026 18:42
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.

3 participants