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

Add mapping between differents headers names and fields object target #16

Closed
wants to merge 2 commits into from

Conversation

xscorpio972
Copy link

Correction for issue #15

@mminella
Copy link
Member

mminella commented Jun 8, 2015

Unless I'm missing something the new annotation is available only at the method level, yet the mapping functionality for the non-annotation version works at the field level as well. Can we get this updated to support both?

@mminella mminella self-assigned this Jun 15, 2015
Copy link

@waasikhaan waasikhaan left a comment

Choose a reason for hiding this comment

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

If i remember correctly PropertyDescriptor will also give us additional property for class. Could it be possible to skip that one in PropertyMatches -> calculateMatches method.

@mdeinum
Copy link
Collaborator

mdeinum commented Sep 19, 2023

This issue hasn't seen much attention. Next to that the same could be achieved (for the mapping) using a StaticColumnNameExtractor and skip the first row of the sheets/xls that is being read. With that I'm going to close this issue.

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