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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: update required fields for submission #10140
chore: update required fields for submission #10140
Conversation
04797e7
to
ba22930
Compare
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.
Looks good! I'm just not sure about this omitBy(input, isEmpty),
....
@@ -56,6 +56,13 @@ export const ArtworkDetailsForm: React.FC = () => { | |||
accessibilityLabel="Title" | |||
/> | |||
<StandardSpace /> | |||
<CategoryPicker<AcceptableCategoryValue | null> |
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.
nitpick: Would it make sense to add null
as a possible value to AcceptableCategoryValue
?
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 wanted to keep my changes to a minimum since this was there already and I only moved it down - I will see if it's a quick change
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.
Leaving it here sounds like a good idea then 馃憤
src/app/Scenes/SellWithArtsy/mutations/updateConsignSubmissionMutation.ts
Outdated
Show resolved
Hide resolved
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.
Looking good! Also agree that the omitBy filtering might not be needed and I added a note about the dimensions fields specifically.
src/app/Scenes/SellWithArtsy/mutations/updateConsignSubmissionMutation.ts
Outdated
Show resolved
Hide resolved
src/app/Scenes/SellWithArtsy/SubmitArtwork/ArtworkDetails/validation.ts
Outdated
Show resolved
Hide resolved
ba22930
to
eb9792f
Compare
3e0a94c
to
41fc71d
Compare
9e3bc20
to
e4a9ba0
Compare
e4a9ba0
to
09a3f17
Compare
This PR resolves ONYX-903
Description
This PR updates the required fields for submissions in-app to be (artist, title and category)
PR Checklist
To the reviewers 馃憖
Changelog updates
Changelog updates
Cross-platform user-facing changes
iOS user-facing changes
Android user-facing changes
Dev changes
Need help with something? Have a look at our docs, or get in touch with us.