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.

Sign up for GitHub

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

Jump to bottom

Dynamic store configuration improvements #1695

Merged
merged 8 commits into from Jun 26, 2019
Merged

Dynamic store configuration improvements #1695

merged 8 commits into from Jun 26, 2019

Conversation

MaceWindu
Copy link
Contributor

@MaceWindu MaceWindu commented Apr 21, 2019 •
edited

Fix #1521, fix #1477

  • respect DynamicColumnsStoreAttribute.Configuration property (before it was using first dictionary with such attribute)
  • allow custom storage setter/getter methods using DynamicColumnAccessorAttribute mapping entity attribute instead of dictionary
  • fluent mapper: new method DynamicColumnsStore to configure store dictionary
  • fluent mapper: new method DynamicPropertyAccessors to configure store getters/setters
  • fix type accessor to use current mapping schema instead of default for default values

@MaceWindu MaceWindu added this to the 2.8.0 milestone Apr 21, 2019
@MaceWindu MaceWindu changed the title [WIP] Dynamic store configuration improvements Dynamic store configuration improvements Apr 22, 2019
@MaceWindu MaceWindu mentioned this pull request Apr 22, 2019
Closed
@MaceWindu MaceWindu merged commit ab3ab58 into master Jun 26, 2019
@MaceWindu MaceWindu deleted the dynamic_store_mapping_support branch June 26, 2019 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sdanyliv sdanyliv sdanyliv approved these changes

@igor-tkachev igor-tkachev Awaiting requested review from igor-tkachev igor-tkachev is a code owner

@GonzoSpire GonzoSpire Awaiting requested review from GonzoSpire

Assignees
No one assigned
Labels
area: fluent-mapping area: mapping
Milestone
2.8.0
Development

Successfully merging this pull request may close these issues.

TypeAccessor<T> doesn't respect IMetadataReader Store dynamic columns outside of current entity
2 participants
@MaceWindu @sdanyliv