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] room을 동적 라우팅에서 쿼리문으로 라우팅 변환 1384 #1385

Conversation

contemplation-person
Copy link
Contributor

📌 개요

  • aws 이유로 동적 라우팅 방식에서 쿼리 방식으로 변경

💻 작업사항

  • /party/[roomId] -> /party/room?id=[id] 로 변경

PHJoon and others added 30 commits March 8, 2024 19:31
공통 레이아웃 디자인 적용을 위함
yoouyeon and others added 13 commits April 6, 2024 21:05
…r-api-연결-중-발견된-에러-수정

[Fix] 지원서 확인하기 페이지 url 수정
[test-deploy] 지원서 확인하기 페이지 url 수정
* join, start, leave 버튼 크기 조정
* hr 태그 크기 조정
* 경고 이미지 margin 조정
* 더 진하게 입력창 색상 조정
* NEXT_PUBLIC_CLIENT_ENDPOINT
* NEXT_PUBLIC_PARTY_MANAGE_SERVER_ENDPOINT
…oom의-css-변경

[Ci] party detail room의 css 변경 - 1358
…oom의-css-변경

[Fix] 방 상세내역의 공유버튼 url 수정
…oom의-css-변경

[Fix] join, leave 버튼 위치 수정
@contemplation-person contemplation-person added enhancement New feature or request user 일반 사용자가 사용하는 서비스에서 발생한 issue test deploy test 배포 (개발 중 배포 테스트) labels Apr 11, 2024
@contemplation-person contemplation-person self-assigned this Apr 11, 2024
@contemplation-person contemplation-person changed the base branch from main to test-deploy April 11, 2024 15:31
Copy link
Member

@yoouyeon yoouyeon left a comment

Choose a reason for hiding this comment

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

pr 보니까 수정한 내용을 테스트 배포 브랜치에 merge하고 있는데, 이 브랜치를 기준으로 최신화하게 되면 나중에 6기 작업들 main으로 머지할 때 테섭 배포되어 있는 모집 기능 작업까지 함께 머지될 것 같아서요.

나중에 브랜치 전략을 다시 한번 얘기해봐야 할 것 같긴 한데, 지금 상황에서는 아래 그림처럼 수정사항들은 6기 개발 브랜치에 계속 올리고, 정기적으로 테스트 배포를 하는 식으로 생각을 하고 있었는데, 6기 분들 사이에서는 어떻게 얘기되었는지 궁금합니다.

image

@contemplation-person
Copy link
Contributor Author

contemplation-person commented Apr 12, 2024

pr 보니까 수정한 내용을 테스트 배포 브랜치에 merge하고 있는데, 이 브랜치를 기준으로 최신화하게 되면 나중에 6기 작업들 main으로 머지할 때 테섭 배포되어 있는 모집 기능 작업까지 함께 머지될 것 같아서요.

나중에 브랜치 전략을 다시 한번 얘기해봐야 할 것 같긴 한데, 지금 상황에서는 아래 그림처럼 수정사항들은 6기 개발 브랜치에 계속 올리고, 정기적으로 테스트 배포를 하는 식으로 생각을 하고 있었는데, 6기 분들 사이에서는 어떻게 얘기되었는지 궁금합니다.

image

저희는 윗 기수 분들이 작업하는 팀원 모집 기능이랑 같이 올라가는 것으로 인지하고 있었어요. (BE쪽에서 같이 올라간다고 해서)
그래서 테스트에 머지하고 테스트가 deploy에 머지되는 방향으로 인지하고 있었습니다.

@yoouyeon
Copy link
Member

@contemplation-person 아하 그랬군요...! 사실 저는 따로 올라갈 수도 있다고 알고 있었어서 이부분은 다시 확인하고 알려드릴게요!
BE에서는 같이 배포하기로 했다고 했으니 아마 같이 올라갈 가능성이 높을 것 같은데 맞다면 이번에는 이미 작업하신 내용도 있고 하니 지금 방식처럼 해도 괜찮을 것 같습니다.
지금 상황과는 별개로 지금 브랜치 전략이 좀 복잡하고 헷갈리는것도 맞는 것 같아서 이번 작업 끝나면 이 부분도 좀 손봐야 할 것 같긴 하네요 ㅎㅎ
알려주셔서 감사합니다!

@contemplation-person contemplation-person changed the base branch from test-deploy to 6th_party April 16, 2024 04:06
@contemplation-person contemplation-person merged commit 034ed4b into 6th_party Apr 16, 2024
1 check passed
@contemplation-person contemplation-person deleted the Feat-room을-동적-라우팅에서-쿼리문으로-라우팅-변환-1384 branch April 16, 2024 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request test deploy test 배포 (개발 중 배포 테스트) user 일반 사용자가 사용하는 서비스에서 발생한 issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] room을 동적 라우팅에서 쿼리문으로 라우팅 변환
7 participants