Skip to content

BTS-2158 | /_api/import: Only allow valid type parameter values #21815

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 26, 2025

Conversation

Simran-B
Copy link
Contributor

@Simran-B Simran-B commented Jun 20, 2025

Scope & Purpose

An omitted type parameter or one with an empty value is valid in addition to document/array/list/auto, but everything else should raise an error

  • 💩 Bugfix
  • 🍕 New feature
  • 🔥 Performance improvement
  • 🔨 Refactoring/simplification

Checklist

  • Tests
    • Regression tests
    • C++ Unit tests
    • integration tests
    • resilience tests
    • Manually tested
  • 📖 CHANGELOG entry made
  • 📚 documentation written (release notes, API changes, ...)
  • Backports
    • Backport for 3.12.0: (Please link PR)
    • Backport for 3.11: (Please link PR)
    • Backport for 3.10: (Please link PR)

Related Information

(Please reference tickets / specification / other PRs etc)

  • Docs PR:
  • Enterprise PR:
  • GitHub issue / Jira ticket:
  • Design document:

An omitted type parameter or one with an empty value is valid in addition to document/array/list/auto, but everything else should raise an error
@Simran-B Simran-B self-assigned this Jun 20, 2025
@Simran-B Simran-B added this to the 3.12 milestone Jun 20, 2025
@cla-bot cla-bot bot added the cla-signed label Jun 20, 2025
Copy link
Member

@neunhoef neunhoef left a comment

Choose a reason for hiding this comment

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

LGTM

@Simran-B Simran-B merged commit ff2fc94 into devel Jun 26, 2025
1 of 2 checks passed
@Simran-B Simran-B deleted the bug-fix/import-api-type-check branch June 26, 2025 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants