Add a switch to turn off auto-mappings #993

Closed
gunnarmorling opened this Issue Dec 12, 2016 · 6 comments

Comments

Projects
None yet
3 participants
@gunnarmorling
Member

gunnarmorling commented Dec 12, 2016

Follows up on #60.

@gunnarmorling gunnarmorling added this to the 1.2.x milestone Jan 11, 2017

@gunnarmorling

This comment has been minimized.

Show comment
Hide comment
@gunnarmorling

gunnarmorling Jan 11, 2017

Member

Assigned this to 1.2 as discussed during this week's hangout.

Member

gunnarmorling commented Jan 11, 2017

Assigned this to 1.2 as discussed during this week's hangout.

@agudian agudian modified the milestones: 1.2.0.Beta1, 1.2.x Jan 11, 2017

@gunnarmorling gunnarmorling modified the milestones: 1.2.0.Beta1, 1.2.x Feb 2, 2017

@gunnarmorling gunnarmorling changed the title from Should we have a switch to turn off auto-mappings to Add a switch to turn off auto-mappings Feb 2, 2017

@gunnarmorling

This comment has been minimized.

Show comment
Hide comment
@gunnarmorling

gunnarmorling Feb 2, 2017

Member
  • Add the switch (processor option? on the mapper level?)
  • Doc update
Member

gunnarmorling commented Feb 2, 2017

  • Add the switch (processor option? on the mapper level?)
  • Doc update
@filiphr

This comment has been minimized.

Show comment
Hide comment
@filiphr

filiphr Feb 5, 2017

Member

Just as a note the fix for #1011 relies on the automapping to create mapping for nested target properties. We need to make sure that the switch is only applied to the correct place

Member

filiphr commented Feb 5, 2017

Just as a note the fix for #1011 relies on the automapping to create mapping for nested target properties. We need to make sure that the switch is only applied to the correct place

@gunnarmorling

This comment has been minimized.

Show comment
Hide comment
@gunnarmorling

gunnarmorling Jun 2, 2017

Member

Hey @filiphr, is this option somewhere described in the docs? I cannot seem to find any mention of it (only the SPI).

Member

gunnarmorling commented Jun 2, 2017

Hey @filiphr, is this option somewhere described in the docs? I cannot seem to find any mention of it (only the SPI).

@filiphr

This comment has been minimized.

Show comment
Hide comment
@filiphr

filiphr Jun 2, 2017

Member

@gunnarmorling not in the docs, but it is in the Javadoc Mapper#disableSubMappingMethodsGeneration

Member

filiphr commented Jun 2, 2017

@gunnarmorling not in the docs, but it is in the Javadoc Mapper#disableSubMappingMethodsGeneration

@gunnarmorling

This comment has been minimized.

Show comment
Hide comment
@gunnarmorling

gunnarmorling Jun 2, 2017

Member

Ok, thanks for clarifying, @filphr. I've filed #1219 for adding it to the ref guide, too.

Member

gunnarmorling commented Jun 2, 2017

Ok, thanks for clarifying, @filphr. I've filed #1219 for adding it to the ref guide, too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment