Skip to content

Implement SurrogateSelector #1693

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
Merged
merged 3 commits into from
May 14, 2018
Merged

Implement SurrogateSelector #1693

merged 3 commits into from
May 14, 2018

Conversation

hazzik
Copy link
Member

@hazzik hazzik commented May 14, 2018

  • Supports System.Type, System.Reflection.MemberInfo and System.Delegate

Closes #1652

@hazzik hazzik force-pushed the serialization-surrogates branch from b31fe84 to cc13fec Compare May 14, 2018 10:45
@hazzik
Implement SurrogateSelector
ac3a637 
- Supports System.Type, System.Reflection.MemberInfo and System.Delegate

Closes nhibernate#1652
@hazzik hazzik force-pushed the serialization-surrogates branch from cc13fec to ac3a637 Compare May 14, 2018 10:53
@hazzik hazzik added this to the 5.2 milestone May 14, 2018
hazzik
hazzik commented May 14, 2018
View reviewed changes
@hazzik hazzik requested a review from fredericDelaporte May 14, 2018 12:01
fredericDelaporte
fredericDelaporte reviewed May 14, 2018
View reviewed changes
fredericDelaporte
fredericDelaporte approved these changes May 14, 2018
View reviewed changes
Copy link
Member

@fredericDelaporte fredericDelaporte left a comment

Choose a reason for hiding this comment

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

(About SerializableType, better change it in another PR, or even wait for someone asking for it to allow supplying surrogates.)

@fredericDelaporte fredericDelaporte merged commit a407afb into nhibernate:master May 14, 2018
@fredericDelaporte fredericDelaporte mentioned this pull request May 14, 2018
Closed
@hazzik hazzik deleted the serialization-surrogates branch May 16, 2018 14:00
@fredericDelaporte fredericDelaporte mentioned this pull request May 31, 2018
Closed
@hazzik hazzik mentioned this pull request Nov 5, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fredericDelaporte fredericDelaporte fredericDelaporte approved these changes

Assignees
No one assigned
Labels
c: Core p: Minor r: Fixed t: New Feature
Projects
None yet
Milestone
5.2
Development

Successfully merging this pull request may close these issues.
2 participants
@hazzik @fredericDelaporte