Move EF core related building to extension methods #616
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…r in application builder
maurei
changed the title
wisepotato
approved these changes
Nov 6, 2019
|
@wisepotato it seems that default interface members with Either way I've refactored the extensions methods to no longer need the internals of |
Merged
wisepotato
pushed a commit
that referenced
this pull request
Nov 11, 2019
* Fix/deviating dbset name (#603) * test: expose dbset name bug * fix: deviating dbset name * chore: add launchSettings.json to gitignore * chore: delete launchSettings.json from git * Fix/pagesize (#601) * feat: new startup for test * chore: rename startup class * test: reproduce #599 in test * fix: zero division error * chore: remove redundant DefaultPageSize on IPageService * chore: update .gitignore * fix: typo LoaDatabaseValues (#608) * fix: typo * fix: typo in fix of typo * chore: enable sql logging in JsonApiDotNetCoreExample * Acceptance tests EF inheritance (#610) * chore: create inheritance model + controller * test: create and patch acceptance test * chore: close issue * Improve usability RequestSerializer (#613) * feat: remove inability to use request serializer with unknown type at runtime * fix: typo in test setup * chore: spacing * fix: add IResourceQueryService and IResourceCmdService to DI container in application builder * chore: update comments IRequestSerializer * Move EF core related building to extension methods (#616) * feat: remove inability to use request serializer with unknown type at runtime * fix: typo in test setup * chore: spacing * fix: add IResourceQueryService and IResourceCmdService to DI container in application builder * chore: update comments IRequestSerializer * chore: moved ef core related building to extension methods * chore: spacing, rename file * refactor: remove need for internal modifier * refactor: minor refactor id type retrieval * Hooks discovery detection of implemented hooks issue (#617) * fix: discovery issue related to assemblies * fix: LoaDatabaseValue typo * fix: set EnableResourceHooks to true by default * chore: add comment in code in hooksdiscovery * fix: use IServiceProvider instead of IScopedServiceProvider * fix: discovery unit test with ServiceProvider instead of ScopedServiceProvider * refactor: TResource instead of TEntity * chore: process PR review * chore: move comment to separate line (stylecop) * feat: use camelCase as default (#618) * feat: use camelCase as default * refactor: massive replace all to replace kebab-case with camelCase default * chore: fix remaining unit tests * fix: fix remaining JsonApiDotNetCoreExampleTests * feat: introduce proper usage of WebApplicationFactory in KebabCaseFormatterTests * fix: remaining functional tests * fix: minor changes PR review * fix: camelCase revert in comments * fix: camelCase revert in comments * fix: camelCase revert in comments * fix: camelCase revert in comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR fixes the inability to use auto discovery when setting up a JADNC application with the ef-core minded
AddJsonApi<TDbContext>overload. It does so by moving all EF Core related configuration to extension methods instead of them being hardcoded in the interface ofJsonApiApplicationBuilder. This is also an important step towards decoupling EF core from the core of JADNC.Inspired by https://dotnetcoretutorials.com/2017/01/24/servicecollection-extension-pattern/