Skip to content

Conversation

corenting
Copy link
Contributor

We changed the field name in the API from document_identifier to filename to better align with other API endpoints.

This is not a breaking change: the API does still support the old document_identifier name.

This is not a breaking change: the API does still support the old
`document_identifier` name.
@corenting corenting requested a review from a team as a code owner August 11, 2025 12:11
@codecov-commenter
Copy link

codecov-commenter commented Aug 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.62%. Comparing base (ca5459c) to head (8a9531d).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #153   +/-   ##
=======================================
  Coverage   95.62%   95.62%           
=======================================
  Files           5        5           
  Lines        1234     1234           
=======================================
  Hits         1180     1180           
  Misses         54       54           
Flag Coverage Δ
unittests 95.62% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@corenting corenting added the skip-changelog No changelog check in the CI label Aug 11, 2025
Copy link
Contributor

@sevbch sevbch left a comment

Choose a reason for hiding this comment

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

LGTM!

@corenting corenting merged commit 5febfc2 into master Aug 13, 2025
19 of 20 checks passed
@corenting corenting deleted the corenting/change-byos-public-api-field-name branch August 13, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog No changelog check in the CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants