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

강원대 FE_최서린 4주차 과제 STEP3,4 #71

Open
wants to merge 24 commits into
base: seorinnn
Choose a base branch
from

Conversation

seorinnn
Copy link

안녕하세요 멘토님!
step2, step4 수행하여 커밋하였습니다!

chakra-ui 를 이번 주차 과제로 인해 처음 사용해보았는데,
막상 사용해보니 왜 이런 ui 라이브러리들을 사용하는지 알 것 같은 느낌이 들었습니당

항상 피드백해주셔서 감사합니다 ㅎㅎ

Copy link

@lazy-sky lazy-sky left a comment

Choose a reason for hiding this comment

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

리드미 업데이트를 제외하고 저번 pr에서 바뀐 내용이 없는 거 같은데 맞나요?

@seorinnn
Copy link
Author

리드미 업데이트를 제외하고 저번 pr에서 바뀐 내용이 없는 거 같은데 맞나요?

앗 확인해보니 저번 step1 pr에 step2 과제 커밋이 들어갔었네요!!
그렇다면 리드미만 업데이트된 것이 맞습니다...!
현재 step1, step2, step4 과제를 수행한 상황입니다!

@seorinnn seorinnn changed the title 강원대 FE_최서린 4주차 과제 STEP2 강원대 FE_최서린 4주차 과제 STEP3,4 Jul 21, 2024
@seorinnn
Copy link
Author

seorinnn commented Jul 21, 2024

step3 수행하여 커밋 추가하였습니다! ㅎㅎ

그런데 step2 과제 Pr에서 머지되지 않은 상태로 이후 step3 커밋을 추가하였더니

This branch has conflicts that must be resolved
Only those with [write access](https://docs.github.com/articles/what-are-the-different-access-permissions) to this repository can merge pull requests.
Conflicting files
package-lock.json
package.json
src/pages/Purchase/index.tsx

라고 뜹니다

멘토님의 '리드미 업데이트를 제외하고 저번 pr에서 바뀐 내용이 없는 거 같은데 맞나요?' 라는 requested changes 이 남아있어서 그런것일까요..?!

@lazy-sky
Copy link

제 request는 코드에 영향을 주지 않습니다. 깃에서 안내하는대로 컨플릭트를 해결하시고 다시 푸시하시면 됩니다.
image

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