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

Respect class inheritance during component argument conversion #121

Merged
merged 7 commits into from Jun 8, 2023

Conversation

s2b
Copy link
Collaborator

@s2b s2b commented Jan 27, 2023

Currently, you have to define argument conversion behavior per input class name. That means that if you defined a conversion interface that takes a FileInterface as input, you need to attach this interface to File, FileReference, ProcessedFile in order to cover all variants.

This patch inspects the class structure of the input object and chooses the appropriate conversion automatically.

@s2b s2b force-pushed the feature/conversionInterface branch 4 times, most recently from 2859b61 to aecf146 Compare January 27, 2023 15:39
@s2b s2b changed the title Feature/conversion interface Respect class inheritance during component argument conversion Jan 27, 2023
@s2b s2b force-pushed the feature/conversionInterface branch from aecf146 to d1a2a84 Compare January 27, 2023 15:41
@s2b s2b requested a review from Atomschinken January 27, 2023 15:47
@s2b s2b force-pushed the feature/conversionInterface branch from 4f15855 to 85a969a Compare March 27, 2023 14:40
@s2b s2b force-pushed the feature/conversionInterface branch from 09751fd to 1825217 Compare June 8, 2023 08:40
@s2b s2b added the 3.7.0 label Jun 8, 2023
@s2b s2b force-pushed the feature/conversionInterface branch from 1825217 to b33cac3 Compare June 8, 2023 09:18
@s2b s2b merged commit 433e7e2 into master Jun 8, 2023
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants