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.

Sign up for GitHub

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

Jump to bottom

Allow to explictly disable the inventory update via property #253

Closed
odrotbohm opened this issue Jan 16, 2019 · 2 comments
Closed

Allow to explictly disable the inventory update via property #253

odrotbohm opened this issue Jan 16, 2019 · 2 comments
Assignees
@martinmo
Milestone
7.2.2

Comments

@odrotbohm
Copy link
Member

Introduce salespoint.inventory.disable-updates to completely disable the inventory update.
@odrotbohm odrotbohm self-assigned this Jan 16, 2019
@odrotbohm odrotbohm added this to the 7.2 milestone Jan 16, 2019
@martinmo martinmo assigned martinmo and unassigned odrotbohm Sep 26, 2019
martinmo added a commit that referenced this issue Oct 14, 2019
@martinmo
#253 - Add property to disable inventory updates (WIP).
47191ed
@martinmo
Copy link
Contributor

I guess this issue should be postponed to 7.2.1.

@odrotbohm odrotbohm modified the milestones: 7.2, 7.3, 7.2.1 Oct 14, 2019
odrotbohm pushed a commit that referenced this issue Nov 26, 2019
@martinmo @odrotbohm
#253 - Introduce salespoint.inventory.disable-updates.
d5bd7d0 
We introduced a configuration property salespoint.inventory.disable-updates to allow to prevent the registration of the event listener that updates the inventory for all order related events.
odrotbohm pushed a commit that referenced this issue Nov 26, 2019
@martinmo @odrotbohm
#253 - Introduce salespoint.inventory.disable-updates.
cbed39c 
We introduced a configuration property salespoint.inventory.disable-updates to allow to prevent the registration of the event listener that updates the inventory for all order related events.
@odrotbohm odrotbohm modified the milestones: 7.3, 7.2.2 Nov 26, 2019
@odrotbohm
Copy link
Member Author

I decided to go with a more drastic approach by simply removing the entire bean from the application context if the property is set. Merged for 7.2.x as well as it doesn't affect existing applications.

@JonasDeepe JonasDeepe mentioned this issue Dec 12, 2019
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@martinmo martinmo

Labels
None yet
Projects
None yet
Milestone
7.2.2
Development

No branches or pull requests
2 participants
@odrotbohm @martinmo