-
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
Feature/megaphone #36
Conversation
Deploy preview for unifest-web-deployment ready! ✅ Preview Built with commit a02716b. |
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.
megaphon 수정하고서 pull request 승인할게요
export function Megaphone({ boothId }: { boothId: number }) { | ||
const accessToken = useAuthStore((state) => state.accessToken); | ||
const [booth, setBooth] = useState<Booth>(); | ||
const getAuthBooth = useAuthFetch(() => getBoothDetail(boothId)); |
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.
디테일 보여주는 getBoothDetail이 그 때 api 상에 문제가 있어서 사용을 못했던 것으로 기억합니다 ㅠ
제가 알아서 수정해볼게요
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.
getBoothDetail 문제 수정된 것으로 확인
useEffect(() => { | ||
const getBoothListEffect = async () => { | ||
const data = await getAuthBooth(); | ||
if (data) { |
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.
response.data 자체가
{
code: string;
message: string;
data: json
}
형태로 날라오기에
const { data } = await getAuthBoth(); 로 바꿔야 합니다
boothId: number, | ||
msg: { msgBody: string }, | ||
) => { | ||
const response = await fetch(`${API_URL}/booths/${boothId}/announcement`, { |
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.
현재 버전에서는 ${API_URL}/megaphone
로 해야만 동작
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
megaphone 페이지, 기능 추가
메가폰 페이지로 이동하기 전 페이지인 웨이팅 페이지가 존재하지않아
따로 메가폰 페이지로 이동하는 인터렉션은 추가하지않음.
'megaphone/[boothid]' url로 이동하여 확인 가능.