Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

Paging infrastructure implemented through specifications. #151

Merged
merged 3 commits into from
Nov 9, 2018
Merged

Conversation

fiseni
Copy link
Contributor

@fiseni fiseni commented Oct 20, 2018

Paging criterias added in BaseSpecification.
New paginated specification added - CatalogFilterPaginatedSpecification.
EFRepository cleaned up and paging implementation added.
Usage of the new paging infrastructure in CatalogService (Web and WebRazor).

…ion added (CatalogFilterPaginatedSpecification) . EFRepository cleaned up and paging implementation added. Usage of the new paging infrastructure in CatalogService (Web and WebRazor).
@dnfclas
Copy link

dnfclas commented Oct 20, 2018

CLA assistant check
All CLA requirements met.

@rjs123431
Copy link

hi, code looks good. can we include sorting here?

- Evaluation of the specification placed in separate class.
@fiseni
Copy link
Contributor Author

fiseni commented Nov 4, 2018

Here you go, OrderBy and OrderByDescending are implemented.
Besides this, GetMin and GetMax methods can be added in the repository in order to evaluate them as part of the query.

@ardalis
Copy link
Collaborator

ardalis commented Nov 9, 2018

looks great, thanks!

@ardalis ardalis merged commit 04530db into dotnet-architecture:master Nov 9, 2018
diegoszch added a commit to diegoszch/eShopOnWeb that referenced this pull request Nov 16, 2018
Paging infrastructure implemented through specifications. (dotnet-architecture#151)
alscoundrel pushed a commit to alscoundrel/eShopOnWeb that referenced this pull request Feb 20, 2020
…hitecture#151)

* Paging criterias added in BaseSpecification. New paginated specification added (CatalogFilterPaginatedSpecification) . EFRepository cleaned up and paging implementation added. Usage of the new paging infrastructure in CatalogService (Web and WebRazor).

* Use IReadOnlyList<T> instead of List<T> as return type from repositories.

* - Ordering possibility added in the specification.
- Evaluation of the specification placed in separate class.
alscoundrel pushed a commit to alscoundrel/eShopOnWeb that referenced this pull request Feb 20, 2020
…hitecture#151)

* Paging criterias added in BaseSpecification. New paginated specification added (CatalogFilterPaginatedSpecification) . EFRepository cleaned up and paging implementation added. Usage of the new paging infrastructure in CatalogService (Web and WebRazor).

* Use IReadOnlyList<T> instead of List<T> as return type from repositories.

* - Ordering possibility added in the specification.
- Evaluation of the specification placed in separate class.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants