-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add API endpoints for default delivery channels for a customer #742
Add API endpoints for default delivery channels for a customer #742
Conversation
a841fd2
to
01b9db7
Compare
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.
Looking at this branch in IDE it might be useful to add new namespaces under API.Features.DeliveryChannels.Requests
. As-is there's a big list and almost duplicate requests for DeliveryChannelPolicy and DefaultDeliveryChannel
e.g. as-is
API/
Features/
DeliveryChannels/
Requests/
CreateCustomerDefaultDeliveryChannel.cs
CreateDeliveryChannelPolicy.cs
DeleteCustomerDefaultDeliveryChannel.cs
DeleteDeliveryChannelPolicy.cs
GetCustomerDefaultDeliveryChannel.cs
...
could be
API/
Features/
DeliveryChannels/
Requests/
Defaults/
CreateCustomerDefaultDeliveryChannel.cs
DeleteCustomerDefaultDeliveryChannel.cs
GetCustomerDefaultDeliveryChannel.cs
Policies/
CreateDeliveryChannelPolicy.cs
DeleteDeliveryChannelPolicy.cs
...
Also general question on naming - do we need "Customer" in controller name, and all of the requests? Would CreateDefaultDeliveryChannel
etc suffice?
src/protagonist/API/Features/DeliveryChannels/CustomerDefaultDeliveryChannelsController.cs
Outdated
Show resolved
Hide resolved
src/protagonist/API/Features/DeliveryChannels/CustomerDefaultDeliveryChannelsController.cs
Outdated
Show resolved
Hide resolved
src/protagonist/API/Features/DeliveryChannels/Requests/CreateCustomerDefaultDeliveryChannel.cs
Outdated
Show resolved
Hide resolved
src/protagonist/API/Features/DeliveryChannels/Requests/CreateCustomerDefaultDeliveryChannel.cs
Outdated
Show resolved
Hide resolved
src/protagonist/API/Features/DeliveryChannels/Requests/CreateCustomerDefaultDeliveryChannel.cs
Outdated
Show resolved
Hide resolved
src/protagonist/API/Features/DeliveryChannels/Requests/UpdateCustomerDefaultDeliveryChannel.cs
Outdated
Show resolved
Hide resolved
src/protagonist/API/Features/DeliveryChannels/Requests/UpdateCustomerDefaultDeliveryChannel.cs
Outdated
Show resolved
Hide resolved
src/protagonist/API/Features/DeliveryChannels/Requests/UpdateCustomerDefaultDeliveryChannel.cs
Outdated
Show resolved
Hide resolved
src/protagonist/API/Features/DeliveryChannels/Requests/CreateCustomerDefaultDeliveryChannel.cs
Outdated
Show resolved
Hide resolved
src/protagonist/API/Features/DeliveryChannels/Requests/UpdateCustomerDefaultDeliveryChannel.cs
Outdated
Show resolved
Hide resolved
5992e3d
to
3e99ac2
Compare
3e99ac2
to
c77e764
Compare
.../API.Tests/Features/DeliveryChannels/Validation/HydraDefaultDeliveryChannelValidatorTests.cs
Outdated
Show resolved
Hide resolved
src/protagonist/API/Features/DeliveryChannels/CustomerDefaultDeliveryChannelsController.cs
Outdated
Show resolved
Hide resolved
...protagonist/API/Features/DeliveryChannels/Validation/HydraDefaultDeliveryChannelValidator.cs
Show resolved
Hide resolved
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.
LGTM - repository test project referencing API project seems off though.
Resolves #720
This pull requests adds API endpoints to
GET
,GET by Id
,PUT
,POST
andDELETE
default delivery channels