Skip to content

Conversation

jake-valsamis
Copy link
Contributor

  • add get_data_asset_file_urls and get_result_file_urls
  • add deprecation warnings for get_data_asset_file_download_url and get_result_file_download_url
  • bump min server version

Copy link

@Copilot 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.

Pull Request Overview

This PR adds Code Ocean version 4.0 functionality with new URL generation methods for both data assets and computation results. The changes deprecate existing single-URL methods in favor of new methods that provide both view and download URLs.

  • Adds new get_data_asset_file_urls and get_result_file_urls methods that return both view and download URLs
  • Deprecates existing get_data_asset_file_download_url and get_result_file_download_url methods with warnings
  • Updates minimum server version requirement to 4.0.0

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/codeocean/folder.py Adds FileURLs dataclass and deprecation warning to DownloadFileURL
src/codeocean/data_asset.py Adds get_data_asset_file_urls method and deprecates get_data_asset_file_download_url
src/codeocean/computation.py Adds get_result_file_urls method and deprecates get_result_file_download_url
src/codeocean/client.py Updates minimum server version to 4.0.0 and adds custom metadata support
src/codeocean/custom_metadata.py Adds new custom metadata schema functionality
src/codeocean/capsule.py Adds comprehensive App Panel support with new dataclasses and methods

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codeocean codeocean deleted a comment from Copilot AI Sep 1, 2025
@codeocean codeocean deleted a comment from Copilot AI Sep 1, 2025
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