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
Fix publish and available for purchase behaviour #780
Fix publish and available for purchase behaviour #780
Conversation
<div className={classes.icon} onClick={handleCloseIconClick}> | ||
<img | ||
src={closeIcon} | ||
alt="" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would add close icon
or something similar as alt
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tests are failing
}: ProductAvailabilityArgs) => ({ | ||
isAvailable: isAvailableForPurchase, | ||
productId, | ||
startDate: availableForPurchase |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
startDate
should be set to null if isAvailableForPurchase
is set to true,
(or maybe it should be changed in ProductUpdatePage)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually not. The idea behind this PR is because this piece of logic was changed for 2.11 - when sending a date for available / published the isAvailable is also to be set to true
…tps://github.com/mirumee/saleor-dashboard into Fix-publish-and-available-for-purchase-behaviour
I want to merge this change because it fixes sending
isPublished
andisAvailable
data to the api when creating / updating product, category or page.SALEOR-1317
also fixes
SALEOR-1344
DateVisibilitySelector
component to handle hide / display logic for the date input and labelsPR intended to be tested with API branch:
use_is_visible_to_determine_if_product_is_publish_for_2_11
Screenshots
Pull Request Checklist
Test environment config
API_URI=https://test-deployment.api.saleor.rocks/graphql/