Bill Wagner
4878101106
Refactor and update to C# 7 constructs
Controllers/BasketController:
Use Expression Bodied Members.
Delete() should not be 'fire and forget'
Controllers/HomeController:
Expression bodied constructor
IntegrationEvents/EventHandling/ProductPriceChangedIntegratedEventHandler:
Use valueTask to minimize allocations for 'fast path' tasks. One might
believe that
most users would not have the changing item in the basket at any given
time.
Using ValueTask instead of Task, this minimizes allocations to only when
async
work must actually take place.
IntegrationEvents/EventHandling/ProductPriceChangedIntegratedEvent:
Make the properties ReadOnly
Model/Basket:
Use property initializer
Model/RedisBasketRepository:
Remove redundant ToString()
Use elvis operator instead of statement null check
Startup:
remove redundant ToString()
7 years ago
Cesar De la Torre
b1c0c72ec7
CreateOrder refactored so return code lines are more compact
7 years ago
Cesar De la Torre
17b3e0f14d
Temporal change of DefaultCacheDuration for the Health Checks so it is shorter for a demo. Need to add overloads at the Health Check library so this is more flexible.
7 years ago
Cesar De la Torre
e74acc62e5
Update README.md
7 years ago
Eduard Tomas
11495156e1
Merge branch 'dev'
7 years ago
Eduard Tomas
0d7e6a16aa
Merge branch 'refactor-event-bus' into dev
7 years ago
Eduard Tomas
79f8f1b949
EventBus refactor.
Instead to register EventHandlers we register Func<EventHandlers> which solves scope problems (having transient/scoped objects owned by singletons)
7 years ago
David Britch
c0f919af4e
Updated Xamarin eBook to preview edition.
7 years ago
David Britch
dc84b09172
Updated Xamarin book and cover.
7 years ago
Cesar De la Torre
4b1467a7b9
Fixing the CreateOrderCommand so it is 100% immutable. It shouldn't have the AddOrderItem() method. In any case, it was not really used but in the tets, since this Command is serialized in the client side, then deserialized in the service level.
7 years ago
Eduard Tomas
829f13aff0
Updated healthcheck lib
7 years ago
Cesar De la Torre
1df0019981
Minor comment typo changeed
7 years ago
Cesar De la Torre
5199d66070
Updated new version of Guide/eBook, v0.5, many changes and new title.
7 years ago
Cesar De la Torre
6f8cd174c7
Refactoring to better terms
7 years ago
Cesar De la Torre
8dc0761d7f
Removing the BasketUrl from the Ordering microservice since it is not used and it shouldn't be used in any case. Don't want synchronous Http communication between the microservices which would impact resiliency. Http only between the client apps and the first level of microservices (or API Gateways, if we had them).
7 years ago
Cesar De la Torre
ab45bb9772
Removing the BasketUrl from the Ordering microservice since it is not used and it shouldn't be used in any case. Don't want synchronous Http communication between the microservices which would impact resiliency. Http only between the client apps and the first level of microservices (or API Gateways, if we had them).
7 years ago
Cesar De la Torre
0e37f421d8
Merge pull request #180 from dotnet-architecture/revert-178-use-IEnumerable-dynamic
Revert "Fix API signatures for collections"
7 years ago
Cesar De la Torre
ed7970802a
Revert "Fix API signatures for collections"
7 years ago
Cesar De la Torre
ebc4de22ad
Merge pull request #178 from BillWagner/use-IEnumerable-dynamic
Fix API signatures for collections
7 years ago
Bill Wagner
995cc890ff
Fix API signatures for collections
The methods that return collections should return
Task<IEnumerable<dynamic>> not Task<dynamic>
7 years ago
David Britch
2578bc0099
Merge pull request #176 from dotnet-architecture/xamarin
Unit tests
7 years ago
David Britch
dfe45883b2
Merge branch 'master' into xamarin
7 years ago
Cesar De la Torre
87846532df
Update README.md
7 years ago
Cesar De la Torre
f3ef118d56
New eBook Cover updated
7 years ago
Cesar De la Torre
d5833277c0
Update README.md
7 years ago
Cesar De la Torre
2785dd7932
Merge pull request #175 from dotnet-architecture/ardalis/typo-patch
Cleaning up and fixing typos
7 years ago
David Britch
9f63e83849
Refactored unit tests.
7 years ago
Ramón Tomás
1a5b6e22e4
Fix: rabbitmq and redis image names were not correctly set in docker compose windows file
Redis and rabbitmq images now are not created when executing built. We force to pull them from docker hub.
7 years ago
David Britch
3692f995b2
Validation unit tests updated.
7 years ago
David Britch
bc06fcd9a2
Unit tests added.
7 years ago
Eduard Tomas
76a94761dd
windows images named differently than linux
7 years ago
RamonTC
9c6b54a7fc
Merge pull request #167 from andrelmp/dev
Upgrade to angular-cli and Angular 4
7 years ago
David Britch
92933928f9
View model unit tests added.
7 years ago
Eduard Tomas
46a030b095
Merge branch 'reviews/rabbitmq'
7 years ago
Steve Smith
a8536b0f3e
Cleaning up and fixing typos
7 years ago
Cesar De la Torre
4f3438903e
Update README.md
7 years ago
Cesar De la Torre
2c285d7b0b
Update README.md
7 years ago
Cesar De la Torre
12c544320d
Merge pull request #173 from GeertVL/master
Fixed some minor typos
7 years ago
David Britch
299a3b74a4
Formatting changes.
7 years ago
Unai Zorrilla Castro
f071ee7fce
Fix bug with queue names
7 years ago
Geert Van Laethem
f7575c1fe0
Merge branch 'master' of https://github.com/dotnet/eShopOnContainers
7 years ago
Geert Van Laethem
3d8fadece4
Fixed two typos to the filter
7 years ago
David Britch
3bcb357fd4
Merge branch 'master' into xamarin
7 years ago
Unai Zorrilla Castro
44e39a685c
Added IRabbitMQPersisterConnection and more resilient work on rabbitmq event bus
7 years ago
David Britch
8a0d5cb6dd
Removed unused namespaces.
7 years ago
David Britch
6b97041d1d
Formatting changes.
7 years ago
David Britch
00a44e1ea3
Removed unused method.
7 years ago
Eduard Tomas
101fc58418
Solves error building SPA on linux host due to case error
7 years ago
David Britch
47bbc722cb
Merge pull request #171 from dotnet/xamarin
Xamarin Client: Replaced Unity with Autofac. Fixes #147
7 years ago
Eduard Tomas
c85e880dad
Catalog.API methods following more rest conventions:
Create/Update routed by POST/PUT & Location header returned
A new GET endpoint for returning single item by id created to honour Location header of previous methods.
7 years ago