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

[Task] 팔로우 팔로워 기능 구현 #419

Merged
merged 39 commits into from
Jan 28, 2024
Merged

Conversation

wade3420
Copy link
Collaborator

🤔 해결하려는 문제가 무엇인가요?

#397
#396
#384

🎉 변경 사항

  • 타인 프로필 검색 기능
  • 타인 프로필
  • 팔로우 검색 페이지
  • 팔로우 검색 페이지에서 타인프로필로 라우팅
  • 타인 프로필에서 팔로우 팔로잉

🙏 여기는 꼭 봐주세요!

  • 팔로잉 기능 구현 사항 모두 구현했습니다 이 브랜치에서 테스트 후에 문제 없으면 develop 으로 머지할게요

사용 방법

🌄 스크린샷

📚 참고

wade3420 and others added 30 commits January 23, 2024 23:56
[Task] 팔로우 미션 리스트 api , 팔로우 멤버 리스트 msw 연결
[Feat] 닉네임 검색 페이지 구현
@wade3420 wade3420 self-assigned this Jan 28, 2024
Copy link

vercel bot commented Jan 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
10mm-client-web ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 28, 2024 8:13am

Copy link
Contributor

@@ -3,25 +3,29 @@ import FollowSummary from '@/app/home/FollowSummary';
import MissionList from '@/app/home/MissionList';
import { flex } from '@styled-system/patterns';

import { useProfileId } from './ProfileIdProvider';
interface ProfileContentProps {
Copy link
Collaborator

Choose a reason for hiding this comment

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

좀 사소한 것인것 같지만
FollowContent에 ProfileContentProps를 받는것 같아 의도된 것일지 궁금해요~!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

이름 바꾸다가 저 Props는 이름 못바꾼듯합니다 우선 머지하고 여기 리뷰 반영하는 pr 다시 열게요!

@@ -30,3 +40,42 @@ const listCss = flex({
flexDirection: 'column',
gap: '8px',
});

export function MissionFollowListInner({ followId }: { followId: number }) {
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 Author

Choose a reason for hiding this comment

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

이것도 따로 리팩토링해야하는 부분입니다 TODO로 남겨둘게요!

export default function Home() {
const [followData, setFollowData] = useState<FollowDataState>(null);
Copy link
Collaborator

Choose a reason for hiding this comment

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

홈에서 follow를 알고 있어야 하는게 몬가몬가 한데 나중에 같이 이야기 하시죵~

@wade3420 wade3420 merged commit 84eb869 into develop Jan 28, 2024
3 checks passed
@wade3420 wade3420 deleted the task/follow-follower branch January 28, 2024 08:13
Copy link
Contributor

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.

3 participants