Comments (7)
We have created a plan regarding this issue.
AutoFilterer dependency will be moved to a library named EasyRepository.EFCore.AutoFilterer by applying correct abstraction techniques in future versions.
While creating this library, a document will be created for the integration of new libraries and developers will be able to add new libraries by reading this document.
AutoFilterer is a great library developed by @enisn that generates automatic queries.
With the improvements we will make, the AutoFilterer library will be added.
We will ask you for help by opening an issue for other libraries.
See you in the future my friend. :D
from easyrepository.efcore.
If you'd like to review my code as I am working on it (currently it is complete), here is the branch on my github repo:
https://github.com/jeffward01/EasyRepository.EFCore/tree/explicit-enums-pagination
I am going to do my best to not introduce any additional packages. There is a chance that the Microsoft.DI framework will not be sufficient, I will need to do more investigation on this to confirm.
If it is not sufficient, I will need to add either:
- Scrutor: Scans assemblies and makes locating injected assemblies and types mostly easy
or - Stashbox: This is a fast easy to use DI framework, similar to AutoFac, but much much faster, about 700% - 2,500% faster than AutoFac (I don't know why people still use Autofac, or even Windsor shudders at the thought of Windsor) - This will be internally used, the consuming developer will have no interactions with it. Simply, it will be used to resolve target implementations of Paginations/Filters
from easyrepository.efcore.
Question for you:
- If you accept this change, what version should I iterate the package to?
Breaking Change:
There will be a breaking change where the user will no longer have a dependency on AutoFilterer
,
The user will need to perform (2) actions to fix the broken changes
-
The user will need to install their Pagination Library or Filter Library of choice, by using going to nuget.org and installing one of:
EasyRepository.EFCore.AutoFilterer
EasyRepository.EFCore.Gridify
EasyRepository.EFCore.Sieve
- (I don't plan to implement Sieve at first, this will be later. Gridify is more performant, and has all the primary features that Sieve has... But Sieve is very popular, so I should definitely implement this at some point in the near future)
-
Then the user will need to go to the container and add this in the DI Registration:
Old Version:
public void ConfigureServices(IServiceCollection services)
{
// Old Version
services.ApplyEasyRepository<TDbContext>()
// Overload...
services.ApplyEasyRepository<TDbContext>(ServiceLifetime.Scoped);
}
New Version:
public void ConfigureServices(IServiceCollection services)
{
// New Version - AutoFilterer || Example of Overload is below:
services.ApplyEasyRepository<TDbContext>().ApplyEasyRepositoryAutoFilterer();
services.ApplyEasyRepository<TDbContext>(ServiceLifetime.Scoped).ApplyEasyRepositoryAutoFilterer();
// New Version - Gridify || Example of Overload is below:
services.ApplyEasyRepository<TDbContext>().ApplyEasyRepositoryGridify();
services.ApplyEasyRepository<TDbContext>(ServiceLifetime.Scoped).ApplyEasyRepositoryGridify();
// New Version - Sieve || Example of Overload is below:
services.ApplyEasyRepository<TDbContext>().ApplyEasyRepositorySieve();
services.ApplyEasyRepository<TDbContext>(ServiceLifetime.Scoped).ApplyEasyRepositorySieve();
}
// Note: Currently, I plan to throw an exception if => 2 or more of these filters / paginators are registered.
// Possible Feature Request:
// Make more flexible and add configuration so the user can register (2) or more of these filters
Note: Currently, I plan to throw an exception if => 2 or more of these filters / paginators are registered.
- Possible Feature Request: Make more flexible and add configuration so the user can register (2) or more of these filters
from easyrepository.efcore.
Sorry for so many comments, I wanted to make all my requests separate comments for reference ease
Question
I will also update the Wiki. How should we do this?
A.) You grant me access to the Wiki, then I update it
B.) I create a new repository, I push the new wiki to it, then you copy and paste the .md
and bring it into the Wiki.
I think A
is easier, Im not sure if you can grant me 'contributor` only rights with approval, im not sure how the Wiki permissions / roles work on GitHub
Note:
I can open up a new issue for this if you want, I was not sure where to put this request...
from easyrepository.efcore.
It's a nice long topic.
We have a static connection with AutoFilterer.
Different filtering and pagination libraries can be supported. This should be very well planned and existing users should not be affected by it.
I can understand the response object you want for your projects.
On the link you shared, a reference was made to the pagination response object that the clients should see. AutoFilterer does not have to provide this response object. You can edit the Response object according to your needs.
I understand your requests on this matter, but this is not a feature that we will support right away.
You can fork and make any changes you want for your own projects.
I will keep this request and we will evaluate and develop it together with my team. Thanks for your contribution.
from easyrepository.efcore.
This should be very well planned and existing users should not be affected by it.
In spirit of this, what about if I make the changes as outlined, with the difference of:
Autofilterer
will be included and used by default- None of the existing return objects or methods will be updated
- Additional methods will be added to support other 'paginated' responses using
Gridify
,Sieve
, or evenAutoFilterer
which can be configured in the startup class by something like:
// Note: I'm not proposing this literally, this is just a sketch to convey what I mean
services.ApplyEasyRepository<TDbContext>(configOptions => {
configOptions.Pagination = SomeEnum.AutoFilterer
})
Something like this?
You can fork and make any changes you want for your own projects.
Thank you, I will do so.
If it's alright with you, I'd rather keep this discussion open and make the changes for my own projects in-line with your vision and scope, I would massively prefer to make the changes with your vision and scope, rather than with an un-guided vision and scope. If the changes are un-guided, in this sense, the likely-hood of a simple later pull-request being incorporated into the library is much lower as it would require a re-write. I would prefer to avoid re-writes and build it with your vision and scope the first time.
On that note, do you think you can outline some hard requirements for this, and I can get a Scope of Work / Development Plans created that we can discuss? Thank you again for your input and I appreciate your opinion ❤️
I agree with you - I dislike breaking changes and its best to avoid them
from easyrepository.efcore.
I will keep this request and we will evaluate and develop it together with my team. Thanks for your contribution.
Ah, I just saw this bit about the team - Fair enough!
Let me know if I could be of further assistance by tagging me in an issue.
I will stand-down on the Pagination and Filter topic and wait for you and your team
from easyrepository.efcore.
Related Issues (15)
- ✨[FEATURE] Implement Abstractions Layer
- ✨ [FEATURE] Unit Of Work
- ✨ [FEATURE] Specification pattern must be supported (Ardalis.Specification)
- 📌 [CUSTOM] Upgrade target framework from dotnet5 to dotnet6
- ✨ [FEATURE] Add 'Enum.AsNoTracking' to all methods instead of using bool values HOT 4
- ✨ [FEATURE] Would like to migrate to 'file scoped' namespaces HOT 2
- ✨ [FEATURE[ Introduce IAsyncEnumerable methods or remove IEnumerable<> as a parameter and change to ICollection<>
- ✨ [FEATURE] UnitOfWork --> EfCore Transactions and Rollbacks? HOT 3
- ✨ [FEATURE] Enable Nullable Reference Types, add null exceptions, add null checks HOT 2
- ✨ [FEATURE] Bulk Delete and Bulk Update with EF.Core 7
- ✨[FEATURE] Implement Add And AddRange operations on generic repository of interface layer
- ✨ [FEATURE] Implement delete operations for generic repository on interface layer
- ✨ [FEATURE] Implement Update Operations for Abstraction Layer
- ✨[FEATURE] Implement Get endpoints for abstraction layer
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from easyrepository.efcore.