Skip to content

Conversation

@jbogard
Copy link
Contributor

@jbogard jbogard commented Nov 16, 2017

No description provided.

@jbogard jbogard added this to the 6.2.1 milestone Nov 16, 2017
@jbogard
Copy link
Contributor Author

jbogard commented Nov 16, 2017

@lbargaoanu thoughts?

@lbargaoanu
Copy link
Contributor

Do we really have to? :) I mean you solved the issue with the DI package, right?

@jbogard
Copy link
Contributor Author

jbogard commented Nov 16, 2017

Kinda solved it, but I think these are two different issues - you can't reset the static mapping, and the ability to configure the DI extension's behavior.

If you wanted to keep the static behavior but still do services.AddAutoMapper then Reset would give you that.

@lbargaoanu
Copy link
Contributor

Maybe some warning in the XML docs? Not intended for production?

@jbogard
Copy link
Contributor Author

jbogard commented Nov 16, 2017 via email

@jbogard jbogard merged commit 146be21 into master Nov 16, 2017
@jbogard jbogard deleted the MapperReset branch November 16, 2017 14:34
@lock
Copy link

lock bot commented May 5, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators May 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants