-
Notifications
You must be signed in to change notification settings - Fork 261
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#11426 dashboard #110
#11426 dashboard #110
Conversation
Added action named GetTopSellingAsync
Added an action named GetPercentOfTotaPayment for sales by payment method (Paypal, demo, etc). Still need development
Returns count of total order status for each one
to decrease calling service aggregated the 3 endpoints into GetDashboard endpoint
@@ -9,6 +9,7 @@ | |||
|
|||
<ItemGroup> | |||
<ProjectReference Include="..\EShopOnAbp.OrderingService.Domain\EShopOnAbp.OrderingService.Domain.csproj" /> | |||
<ProjectReference Include="..\EShopOnAbp.OrderingService.Application.Contracts\EShopOnAbp.OrderingService.Application.Contracts.csproj" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not correct. EntityFrameworkCore layer shouldn't depend on Application.Contracts. See my comment for OrderingServiceRemoteServiceConsts
.
public static class OrderingServiceRemoteServiceConsts | ||
{ | ||
public const string RemoteServiceName = "Ordering"; | ||
public const int Top10 = 10; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the wrong layer for domain-related constants. Constants should be under Domain.Shared especially when you are planning to use them on EntityFrameworkCore layer.
Ordering dashboard implemented
closes abpframework/abp#11426