-
Notifications
You must be signed in to change notification settings - Fork 1
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
4larin/component routing #6
base: master
Are you sure you want to change the base?
Conversation
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.
This looks good as a start but these issues I mentioned needs to be resolved first before I can merge.
on a side note I think the reason changes aren't going through is totally a back end issue.
}, | ||
]} | ||
> | ||
<Input /> |
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.
Why everything here is an input field?
there are different fields we use and some of them are way more complicated, for example but no limited to: description uses markdown input field
<Upload.Dragger | ||
name="heroImage" | ||
action={`${apiUrl}/media/upload`} | ||
listType="picture" | ||
maxCount={5} | ||
multiple | ||
> | ||
<ImageField | ||
value="" | ||
title="" | ||
width={200} | ||
/> | ||
</Upload.Dragger> |
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.
We only have one heroImage and here it's possible for a user to put multiple!
{/* <ShowButton | ||
hideText | ||
size="small" | ||
recordItemId={record.id} | ||
/> | ||
/> */} |
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 did hide this as it's not going to be used for now.
{ label: "Slug", name: "slug", required: true }, | ||
{ label: "CatchPhrase", name: "catchPhrase", required: true }, | ||
{ label: "Title", name: "title", required: true }, |
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.
The second key of these three array objects is "name" but the rest is "title". we have to be consistent with those names so either pick "title" or "name"
No description provided.