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

[Spike] Remove unused order state 'Resumed' #7734

Open
Tracked by #7511
lin-d-hop opened this issue Jun 3, 2021 · 0 comments
Open
Tracked by #7511

[Spike] Remove unused order state 'Resumed' #7734

lin-d-hop opened this issue Jun 3, 2021 · 0 comments
Labels
spike A time-boxed investigation/implementation to access suitability of a tool or feature tech debt

Comments

@lin-d-hop
Copy link
Contributor

What we should change and why (this is tech debt)

This spike is to explore removing Order State = 'Resumed'
While there are occurrences of orders with state = 'Resumed' in the DBs this is a meaningless state to OFN. Please change these orders to state 'complete'.

Context

Spree contained a number of order states that we don't actually need in our flow. On occasion we see orders with these order states and all kinds of weird buggy behaviour ensues.

Impact and timeline

This is a timeboxed spike at 1hr. Please spend no more than 1 hr and report back on how complex it might be to remove the state.
If you find that you can actually complete the task of removing within an hour then feel free to do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spike A time-boxed investigation/implementation to access suitability of a tool or feature tech debt
Projects
Status: All the things 💤
Development

No branches or pull requests

1 participant