Skip to content

Conversation

@fahrigedik
Copy link
Member

@fahrigedik fahrigedik commented Dec 4, 2025

Introduces IsNullable detection in ReflectionHelper and propagates it to PropertyApiDescriptionModel and TypeScript models. Updates logic to consider nullable properties when determining optionality, improving API schema accuracy for nullable reference types.

Description

Resolves #24306 (write the related issue number if available)

How to test it?

test.md

Introduces IsNullable detection in ReflectionHelper and propagates it to PropertyApiDescriptionModel and TypeScript models. Updates logic to consider nullable properties when determining optionality, improving API schema accuracy for nullable reference types.
@fahrigedik fahrigedik added this to the 10.0-patch milestone Dec 4, 2025
@fahrigedik fahrigedik requested a review from maliming December 4, 2025 07:33
maliming and others added 5 commits December 7, 2025 13:17
Adds support for marking properties as nullable by appending '| null' to their type in the model generator. Also refines the logic for determining optional properties to better align with 'isRequired' and 'isNullable' flags.
The isOptionalProperty function now only checks if a property is not required, removing the isNullable condition. This simplifies the logic for determining optional properties.
@sumeyyeKurtulus sumeyyeKurtulus self-requested a review December 12, 2025 10:38
@sumeyyeKurtulus sumeyyeKurtulus merged commit 73f9d9b into rel-10.0 Dec 12, 2025
3 checks passed
@sumeyyeKurtulus sumeyyeKurtulus deleted the issue-24306-v2 branch December 12, 2025 10:39
@JanneHarju
Copy link

Now I found this what might fix that enum nullable probelm. But is this coming to Abp 9.3?

@sumeyyeKurtulus
Copy link
Contributor

Now I found this what might fix that enum nullable probelm. But is this coming to Abp 9.3?

Hello, this change will be effective within the latest patch release for the version 10.

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.

5 participants