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 observer to remove returned/creditmemo products from orders #77

Open
seansan opened this issue Feb 12, 2019 · 0 comments
Open

Add observer to remove returned/creditmemo products from orders #77

seansan opened this issue Feb 12, 2019 · 0 comments

Comments

@seansan
Copy link
Contributor

seansan commented Feb 12, 2019

Add observer to remove returned/creditmemo products from orders

Clerk is a great relational data linker... but when the incoming data is incorrect and returned items are not removed from the order data .... the results can be

  1. many in-statistical recommendations later on
  2. You could argue that Clerk could "learn" what does NOT work well together

Currently returns and changed items are not removed from the sales product sync to Clerk. Our service dept sometimes updates orders adds/removes products before shipping and also customers return products (often 18-22%) ...... so we could have 22% statistical data that should not be there. And more important: the system is neglecting human feedback: people do not want the product.

I hope you see the statistical importance. Recommendations based on for instance 22% insignificant data would not seem very logical. Also imagine the following case: of all 5 occurences of product A and B being ordered in 1 sale, 100% of product B is returned. Maybe we should then not-suggest it.

Maybe an observer can be installed to monitor order changes and credit memos and update Clerk.

The process would then be

  1. order made, sync data
  2. (full) credit memo created, cancel order, create a new 1: sync data
    3 or (partial) credit memo where 1 or 2 items in the order are returned/credited, removev items from the order: syn new data

I hope this helps

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant