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.

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

feat:previewcard #13

Merged
merged 2 commits into from
Jan 17, 2024
Merged

feat:previewcard #13

merged 2 commits into from
Jan 17, 2024

Conversation

JANGHANPYEONG
Copy link
Collaborator

No description provided.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이거 바꾸신 이유가 있나요?

package.json Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

마찬가지로 바꾸신 이유가 있는지 궁금합니다

};

return (
<Card
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cardactionarea 추가해서 동작은 props로 받아오게 하면 좋을 거 같습니다

};

return (
<Card sx={{ width: 200, borderRadius: 16 }}>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

마찬가지로 props로 클릭 시 수행할 function 받아서 cardActionArea 지정하면 좋을 거 같습니다


return null;
} catch (error) {
enqueueSnackbar('유저정보불러오기 실패하였습니다.', { variant: 'error' });
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'사용자 정보를 불러오지 못했습니다'가 더 좋을 거 같습니다

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

assets 폴더에 넣어주시는 게 맞을 거 같습니다

@JANGHANPYEONG JANGHANPYEONG merged commit 62596ae into develop Jan 17, 2024
@JANGHANPYEONG JANGHANPYEONG deleted the feature/previewcards branch January 17, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants