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

Revert "[X] do not apply Bindings if DataType doesnt match (#22056)" #24046

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

PureWeen
Copy link
Member

@PureWeen PureWeen commented Aug 6, 2024

Description of Change

Before

image

After

image

This reverts commit cb0a332.

It caused the following regression
#23989

@PureWeen PureWeen requested a review from a team as a code owner August 6, 2024 18:05
@PureWeen PureWeen requested review from rmarinho and jsuarezruiz and removed request for a team August 6, 2024 18:05
@mattleibow
Copy link
Member

Do we have an issue that tracks the revert of the revert and that is assigned to Stephane?

@PureWeen
Copy link
Member Author

PureWeen commented Aug 6, 2024

Do we have an issue that tracks the revert of the revert and that is assigned to Stephane?

Yea, I didn't revert this on main
So, the issue @davidortinau logged is still valid

@PureWeen PureWeen merged commit 1d82ab0 into release/8.0.1xx-sr8 Aug 6, 2024
20 of 62 checks passed
@PureWeen PureWeen deleted the revert_22056_net8 branch August 6, 2024 21:44
@github-actions github-actions bot locked and limited conversation to collaborators Sep 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants