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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[shopsys] removed preorder and vendor delivery date features #2942

Merged
merged 5 commits into from Dec 7, 2023
Merged

[shopsys] removed preorder and vendor delivery date features #2942

merged 5 commits into from Dec 7, 2023

Conversation

grossmannmartin
Copy link
Member

@grossmannmartin grossmannmartin commented Dec 6, 2023 •
edited by github-actions bot

Q A
Description, reason for the PR These settings were confusing to use as they affected the visibility and availability of the product in unforeseen ways. The Preorder field was also strangely named in the administration (allow overselling), which adds to the confusion.
New feature No
BC breaks Yes
Fixes issues ...
Have you read and signed our License Agreement for contributions? Yes

馃寪 Live Preview:

@grossmannmartin
removed preorder and vendor delivery day feature
b77f53b 
- global stock setting "Days until stocking" was removed
- product's field preorder (Allow Overselling) was removed
- product's field vendorDeliveryDate (Supplier's delivery time) was removed
@grossmannmartin
regenerated graphql schema
42a5539
@grossmannmartin
entity extension test is now skipped
5b673ce 
- solution is proposed in #2919
@grossmannmartin
added upgrade notes
d12cb16
@grossmannmartin
dumped translations
06dd8b3
@grossmannmartin grossmannmartin merged commit 52fa731 into 14.0 Dec 7, 2023
15 of 18 checks passed
@grossmannmartin grossmannmartin deleted the mg-remove-preorder branch December 7, 2023 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TomasLudvik TomasLudvik TomasLudvik approved these changes

@pk16011990 pk16011990 pk16011990 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@grossmannmartin @TomasLudvik @pk16011990