This repository has been archived by the owner on Nov 17, 2023. It is now read-only.
Fix WebSPA catalog pager display #1629
Merged
Conversation
This file contains 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
|
|
rulyotano
added a commit
to rulyotano/eShopOnContainers
that referenced
this pull request
Apr 9, 2021
* Fix ordering test settings * catch error with interceptor * interceptor in mobile bff * top level statements * Prepare to update ng version * Update TS version * Update peerDependencies * Renew packageLock * Upgrade to ng version 9 * Add record types for DTOs. Update ordering DockerFiles to dotnet 5.0 base images. * Fixed call error when filter item is applied * Upgrade node version 10.3 to docker * fix namespace top level statement * net5 * record types * customer basket items setteable * Fix MarketingUrl Port on AppSettings * Update nuget packages for Ordering service. * Fix CampaignDetailFunctionEnabled required when conf isReady * Delete Package lock to update ng version * new PackageLock to update to ng version10 * new PackageLock to update to ng version10 * Change Grpc client creation from a per request approach to a client factory one. * migrate catalog api * Updated to ng version10 * Upgrade to 12 node version * grpc improvements * Top-level statements. Change ordering net standard projects to dotnet 5. * Update logging nuge package dependency. * update net 5 * net 5 webapps * update nuget packages * Migrate Payment.API to dotnet 5 * dockerfiles and update nugets * fix docker file image * update nugets * fix functional tests * remove marketing and locations * update mvc references * update dockerfiles without location & api csprojs * remove campaings from SPA * remove helm deploy marketing locations and builds * delete marketing and locations references * WIP * apigws workflow * paths filter * fix workflow filter path * event name * copy with ps * test basket api workflow * remove uses * fix path filter * add test branch * test registry * test login with cmd * test cmd * test docker login * test echo * echo username * test login * revert login * test login registry endpoint hub.docker.com * Test login without registry variable * Test login registry empty * Fix varible usage in bash * upgrade nuget packages * Fix docker-compose build args * Fix docker-compose push args order * Add checkout * consolidate packages * Fix variables * Add login before build * Fix indentation. * Linux version * fix Breaking change net 5 missing method app.UseDatabaseErrorPage() dotnet/efcore#22687 * Add docker-machine env eval * integration events updated to record types * remove old grpc caller * Docker-machine env dev * sudo * Remove repeated step. * Swamy/remove unused using and refactor the code (dotnet-architecture#1555) * Removed Unused Using and Reorganized the Using * Removed unused using, Reorganized using, moved the class to separate file, removed commented code in Catalog.API * Revert "Removed unused using, Reorganized using, moved the class to separate file, removed commented code in Catalog.API" This reverts commit 34241c4. * Removed unused using and reorganized the using inside "Services" folder * Removed Unused using and reoganized the using * Refactor Webhooks.API * Removed unused using and reorganized using inside Catalog.API * Refactoring * Removed unsed using * Added line break just to differentiate between the messages * Removed unused usings * Simple Refactoring * Get git branch name * Fix get branch step * Uncomment steps * add registry * Fix login * fix cookie policy * update nuget in catalog unit tests * TAG REGISTRY * Fixed Issue dotnet-architecture#1410 (dotnet-architecture#1560) Added IdentityUrlExternal to src\Services\Ordering\Ordering.FunctionalTests\appsettings.json; sorted the properties * remove mobile project * Update env variables. * export variable * Test variable asignation * Fix export commnad * Fix exporting variables when sudo is used * Remove comments * Enable push image * Fix docker-file path for push step. * Add TAG variable to push step. * Update registry path * Change docker login step. * remove load test references * Try sudo docker login in push step. * Update registry value. * Change login method * Refactor yaml * Remove debug traces * test docker hub host * modify registry host url * host url * test docker io * build on pr * test windows build * test login windows * test branch name windows * test windows * test replace branch name pwsh * test * replace feature * working dir and build order cmd * work dir path * work dir * checkout windows * test multiarch step * test docker config env * multiarch * test WIP * WIP * test * test sudo * test docker dir * test * WIP * test * test * test * test * test * test * test * test * test * test cp * test * test * test * test * final * test * test * test * final version * final basket workflow * final version * fix secret ref * basket catalog identity ordering workflows * added github workflows * remove azure pipelines * test builds on branch * Small Refactoring (dotnet-architecture#1558) * final workflows * Updated to aspnet 5 dockerfiles.development for azds * fix restores on docker.develop images * revert host setting * missing identityurlexternal in helm config * Small-Formatting-Refactoring (dotnet-architecture#1564) * Swamy/01jan2021-small-refactoring (dotnet-architecture#1570) * Update Startup.cs * Removed commented code * Update UrlsConfig.cs * Small Refactoring * Removed Commented Code * Small Refactoring * remove windows build * remove trigger on working branch * Removes dotnet.myget.org NuGetPackage feed references (dotnet-architecture#1576) * Removed Nuget package feed * Removed unused System.ValueTuple package * Downloadgraded Microsoft.AspNetCore.Hosting.Abstractions package * Removed unused package * Downgraded System.IO.Compression.ZipFile package * Used Microsoft.AspNetCore.DataProtection.StackExchangeRedis package * Upgrade key packages * Upgrade healthchecks packages * readme actions * Upgrade identity server, fix VS F5 unauthorized_client problem * Swamy/04 jan2021-small-refactoring (dotnet-architecture#1571) * Small Refactoring inside IntegrationEventLogEF * Small Refactoring inside BuildingBlocks folder * Fixes dotnet-architecture#1581: Fixed total decimal place issue and User hover menu display/usability issue. (dotnet-architecture#1582) * Swamy/15jan2021 small refactoring (dotnet-architecture#1580) * Small Refactoring inside Basket Service folder * Small Refactoring * Incorrect ContentPage.ToolbarItems in a TabbedPage (dotnet-architecture#1348) Incorrect ContentPage.ToolbarItems in a TabbedPage * use identity internal network * fix identity url in current helm version * revert archived * clean up solution * Add community code changes since migration start * Remove "obsolete" folder * Remove main branch from action triggers * fix envoy config in helm * update helm chart to use sqlserver 2019 * rerun GitHub Actions * program to top level statement * path filter on pr SPA * added path filter on pr * test path filter on spa * formatting * Bump socket.io from 2.3.0 to 2.4.1 in /src/Web/WebSPA (dotnet-architecture#1588) Bumps [socket.io](https://github.com/socketio/socket.io) from 2.3.0 to 2.4.1. - [Release notes](https://github.com/socketio/socket.io/releases) - [Changelog](https://github.com/socketio/socket.io/blob/2.4.1/CHANGELOG.md) - [Commits](socketio/socket.io@2.3.0...2.4.1) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * include subfolders in filtering path * Updated npm packages. (dotnet-architecture#1594) * Fixes : dotnet-architecture#1599 | Applies patch to dotnet/sdk buster-slim docker image (dotnet-architecture#1601) * Included docker file patch * Used specific npm 6 package version * Updated readme files (dotnet-architecture#1589) * Updated readme files * Updated eShop architecture diagram * Update README.md * marked package updated to latest ver 2.0 (dotnet-architecture#1605) * Feature/add mobile readme (dotnet-architecture#1606) * Readme file added for mobile client * readme file added for mobile client * Update src/Mobile/README.md Co-authored-by: Sumit Ghosh <[email protected]> * eshopOnContainers Architecture document updated Co-authored-by: Sumit Ghosh <[email protected]> * GracePeriodManagerTask renamed in GracePeriodManagerService (dotnet-architecture#1615) * Service Bus persisted connection for ISubscriptionClient (dotnet-architecture#1521) * Fix ordering-signalrhub workflow badge image (dotnet-architecture#1624) * Bump elliptic from 6.5.3 to 6.5.4 in /src/Web/WebSPA (dotnet-architecture#1623) Bumps [elliptic](https://github.com/indutny/elliptic) from 6.5.3 to 6.5.4. - [Release notes](https://github.com/indutny/elliptic/releases) - [Commits](indutny/elliptic@v6.5.3...v6.5.4) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Fix WebSPA pager display (dotnet-architecture#1629) * updated package version for ssri and is-svg (dotnet-architecture#1636) * Fix WebSPA catalog filter (dotnet-architecture#1637) * updated CardType and Enumeration classes (dotnet-architecture#1528) * updated CardType and Enumeration classes * Update per IEvangelist's suggestion * Update src/Services/Ordering/Ordering.Domain/SeedWork/Enumeration.cs Co-authored-by: David Pine <[email protected]> Co-authored-by: Sumit Ghosh <[email protected]> Co-authored-by: David Pine <[email protected]> Co-authored-by: David Sanz <[email protected]> Co-authored-by: Borja García Rodríguez <[email protected]> Co-authored-by: Carlos de Miguel Ferrero <[email protected]> Co-authored-by: Miguel Veloso <[email protected]> Co-authored-by: Borja García <[email protected]> Co-authored-by: Viswanatha Swamy <[email protected]> Co-authored-by: InstanceFactory <[email protected]> Co-authored-by: Sumit Ghosh <[email protected]> Co-authored-by: Hetal Kapadia <[email protected]> Co-authored-by: william-keller <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Nish Anil <[email protected]> Co-authored-by: Sreenivasulu Kalluru <[email protected]> Co-authored-by: alecola <[email protected]> Co-authored-by: Roman Marusyk <[email protected]> Co-authored-by: Konstantin Chuvilyov <[email protected]> Co-authored-by: n-stefan <[email protected]> Co-authored-by: Matt Callahan <[email protected]> Co-authored-by: David Pine <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Hi,
in both WebMVC and WebSPA the catalog pager initially displays:
"Showing 10 of 13 products - Page 1 - 2"
which is correct, 10 products are shown.
However, after clicking "Next":
WebMVC: "Showing 3 of 13 products - Page 2 - 2"
WebSPA: "Showing 10 of 13 products - Page 2 - 2"
The WebSPA pager is incorrect, only 3 products are shown.
This PR aims to fix it.