Skip to content
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

fix(codegen): markup-ext improper cast for nullable value-type dp #18971

Conversation

Xiaoy312
Copy link
Contributor

GitHub Issue (If applicable): closes #18819

PR Type

What kind of change does this PR introduce?

  • Bugfix

What is the current behavior?

Generated xaml for custom MarkupExtension that provide null as value to nullable value-type throws NullReferenceException.

What is the new behavior?

^ no more.

PR Checklist

Please check if your PR fulfills the following requirements:

Other information

@github-actions github-actions bot added the area/code-generation Categorizes an issue or PR as relevant to code generation label Nov 29, 2024
@unodevops
Copy link
Contributor

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18971/index.html

@unodevops
Copy link
Contributor

🤖 Your WebAssembly Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-18971/index.html

@Xiaoy312 Xiaoy312 enabled auto-merge November 30, 2024 00:53
@Xiaoy312 Xiaoy312 disabled auto-merge November 30, 2024 19:28
@Xiaoy312 Xiaoy312 self-assigned this Nov 30, 2024
@Xiaoy312 Xiaoy312 force-pushed the dev/xygu/20241129/markup-ext-nullable-valuetype-invalid-cast branch from 7d2a9d8 to 0ab2847 Compare November 30, 2024 19:34
@Xiaoy312 Xiaoy312 enabled auto-merge November 30, 2024 19:35
@Xiaoy312 Xiaoy312 removed their assignment Nov 30, 2024
@unodevops
Copy link
Contributor

🤖 Your WebAssembly Sample App stage site is ready! Visit it here: https://unowasmprstaging.z20.web.core.windows.net/pr-18971/index.html

@unodevops
Copy link
Contributor

🤖 Your Docs stage site is ready! Visit it here: https://unodocsprstaging.z13.web.core.windows.net/pr-18971/index.html

@unodevops
Copy link
Contributor

⚠️⚠️ The build 149016 has failed on Uno.UI - CI.

@jeromelaban jeromelaban disabled auto-merge December 4, 2024 15:49
@jeromelaban jeromelaban merged commit 3df0b6f into master Dec 4, 2024
112 of 114 checks passed
@jeromelaban jeromelaban deleted the dev/xygu/20241129/markup-ext-nullable-valuetype-invalid-cast branch December 4, 2024 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/code-generation Categorizes an issue or PR as relevant to code generation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can not bind Template Parent's Property in ItemsRepeater's ItemTemplate
5 participants