Skip to content
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 "Ready for Pickup" button on the Order page in Admin Panel which notifies Customer that Order could be picked up #2034

Closed
maghamed opened this issue Feb 11, 2019 · 7 comments
Assignees

Comments

@maghamed
Copy link
Contributor

Add "Ready for Pickup" button to the Order Admin page for orders created with Store Pickup method, click on this button will initiate email notification to the customer if there are enough Qty on a selected source to fulfill the order.

If there are NOT enough Qty to fulfill order the merchant should be notified that he has to proceed with Internal Transition (from Source B -> Source A ) to increase Qty for the Source used in Store Pickup.

@maghamed maghamed added this to the MSI Part III milestone Feb 11, 2019
@TadeuRodrigues TadeuRodrigues self-assigned this Feb 14, 2019
@TadeuRodrigues
Copy link
Member

I'm starting to work.

@maghamed
Copy link
Contributor Author

hey @TadeuRodrigues do you have any result so far? or should we make this ticket available for up for grabs?

@TadeuRodrigues
Copy link
Member

@maghamed yes I have, let me finish some tests than I will send my branch.

novikor added a commit that referenced this issue Mar 28, 2019
…notifies Customer that Order could be picked up #2034.

Added the button itself
@novikor novikor pinned this issue Mar 28, 2019
@novikor novikor unpinned this issue Mar 28, 2019
novikor added a commit that referenced this issue Mar 28, 2019
…notifies Customer that Order could be picked up #2034.

Added check if order is ready for pickup
@novikor
Copy link
Contributor

novikor commented Mar 28, 2019

If there are NOT enough Qty to fulfill order the merchant should be notified that he has to proceed with Internal Transition

@maghamed, could you suggest a way we can implement this behavior?
How and where the notification should look like?
As I know, there are no mechanisms for Internal Transitions, are they?

@TadeuRodrigues
Copy link
Member

@novikor i was working in this issue, let's me send my branch.

@novikor
Copy link
Contributor

novikor commented Mar 29, 2019

@TadeuRodrigues - thanks, let`s try

novikor added a commit that referenced this issue Mar 31, 2019
…notifies Customer that Order could be picked up #2034.

Moved IsOrderReadyForPickup to API level.
Created an init point for NotifyPickup controller.
novikor added a commit that referenced this issue Mar 31, 2019
…notifies Customer that Order could be picked up #2034.

NotifyOrderIsReadyAndShipInterface API;
Implemented NotifyPickup controller;
Added some TODOs
novikor added a commit that referenced this issue Mar 31, 2019
…notifies Customer that Order could be picked up #2034.

Added canShip() check to IsOrderReadyForPickup;
Added TODO
novikor added a commit that referenced this issue Apr 1, 2019
…notifies Customer that Order could be picked up #2034.

Implemented synchronous email sending with custom email template for storepickup.
Added even more TODOs
novikor added a commit that referenced this issue Apr 1, 2019
…notifies Customer that Order could be picked up #2034.

Do not show button is order is not placed using storepickup or it can not be shipped
@ishakhsuvarov
Copy link
Contributor

Closing this ticket now, as initial implementation is already delivered to the store-pickup branch in #2127

@ishakhsuvarov ishakhsuvarov added this to the Store Pickup Support milestone May 30, 2019
ishakhsuvarov pushed a commit that referenced this issue Nov 22, 2019
…notifies Customer that Order could be picked up #2034.

Added the button itself
ishakhsuvarov pushed a commit that referenced this issue Nov 22, 2019
…notifies Customer that Order could be picked up #2034.

Added check if order is ready for pickup
ishakhsuvarov pushed a commit that referenced this issue Nov 22, 2019
…notifies Customer that Order could be picked up #2034.

Moved IsOrderReadyForPickup to API level.
Created an init point for NotifyPickup controller.
ishakhsuvarov pushed a commit that referenced this issue Nov 22, 2019
…notifies Customer that Order could be picked up #2034.

NotifyOrderIsReadyAndShipInterface API;
Implemented NotifyPickup controller;
Added some TODOs
ishakhsuvarov pushed a commit that referenced this issue Nov 22, 2019
…notifies Customer that Order could be picked up #2034.

Added canShip() check to IsOrderReadyForPickup;
Added TODO
ishakhsuvarov pushed a commit that referenced this issue Nov 22, 2019
…notifies Customer that Order could be picked up #2034.

Implemented synchronous email sending with custom email template for storepickup.
Added even more TODOs
ishakhsuvarov pushed a commit that referenced this issue Nov 22, 2019
…notifies Customer that Order could be picked up #2034.

Do not show button is order is not placed using storepickup or it can not be shipped
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants