Skip to content

Conversation

@frano-m
Copy link
Contributor

@frano-m frano-m commented Dec 2, 2025

Ticket

Closes #4603.

Reviewers

@NoopDog.

This pull request updates dependencies and refines table download functionality for several AnVIL catalog entity configurations. The main focus is on ensuring table downloads are consistently enabled and properly configured, while also keeping dependencies current and adding support for new packages.

Dependency updates and additions:

  • Updated several dependencies in package.json, including @databiosphere/findable-ui, next, next-auth, next-mdx-remote, and @next/eslint-plugin-next, to their latest versions for improved stability and compatibility. [1] [2] [3]
  • Added new dependencies gray-matter and slugify to support markdown parsing and string slugification, respectively. Also added an overrides section for the glob package. [1] [2]

Table download configuration improvements:

  • For the consortia, studies, and workspace entity configurations (consortiaEntityConfig.ts, studiesEntityConfig.ts, workspaceEntityConfig.ts), added downloadFilename and enableTableDownload options to the tableOptions object, standardizing table download behavior. [1] [2] [3]
  • Removed the redundant enableDownload property from the listView configuration in the same entity config files, consolidating download controls under tableOptions. [1] [2] [3]

@frano-m frano-m marked this pull request as ready for review December 2, 2025 07:59
@frano-m frano-m requested a review from Copilot December 2, 2025 07:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Contributor

@MillenniumFalconMechanic MillenniumFalconMechanic left a comment

Choose a reason for hiding this comment

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

LGTM, thanks F.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

Update findable-ui to latest version 46.1.3

4 participants