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

✅ [STMT-62] 스터디 상세 정보 조회 API 테스트 추가 및 명세서 작성 #103

Merged
merged 2 commits into from
Mar 26, 2024

Conversation

05AM
Copy link
Collaborator

@05AM 05AM commented Mar 18, 2024

🤔 어떤 방식으로 해결했는지 적어주세요

  • 스터디 상세 정보 조회 테스트 및 명세서를 작성했습니다.

✅ 확인 요망

  • 1.5 사용자 닉네임 유효성 검사에 응답(200)이 공백인데 혹시 의도하신 것인지 확인 부탁 드립니다!
=== 사용자 닉네임 유효성 검사

사용자 닉네임의 유효성을 검사하는 API입니다.

==== GET /api/v1/members/validate-nickname

===== 요청
include::{snippets}/validate_nickname/success/http-request.adoc[]
include::{snippets}/validate_nickname/success/request-headers.adoc[]
include::{snippets}/validate_nickname/success/query-parameters.adoc[]

===== 응답 성공 (200)

===== 응답 실패 (400)

@05AM 05AM added the D-2 2일 전 까지 리뷰해주세요 label Mar 18, 2024
@05AM 05AM self-assigned this Mar 18, 2024
@05AM 05AM requested a review from zxcv9203 as a code owner March 18, 2024 14:05
@github-actions github-actions bot added D-1 1일 전 까지 리뷰해주세요 and removed D-2 2일 전 까지 리뷰해주세요 labels Mar 18, 2024
@zxcv9203
Copy link
Collaborator

헉 이게 왜없지 추가해야할 것 같습니다 😢

Copy link
Collaborator

@zxcv9203 zxcv9203 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. :)

근데 저희 스터디 상세(홈 화면) 페이지 접근시 스터디 멤버가 아닌 회원도 해당 페이지에 접근 가능한가요?
기능 명세서에 제가 못찾는건지 안보여서 알고 계신내용 있으신가요?

@github-actions github-actions bot added D-0 바로 리뷰가 필요해요 and removed D-1 1일 전 까지 리뷰해주세요 labels Mar 19, 2024
@05AM
Copy link
Collaborator Author

05AM commented Mar 21, 2024

근데 저희 스터디 상세(홈 화면) 페이지 접근시 스터디 멤버가 아닌 회원도 해당 페이지에 접근 가능한가요?
기능 명세서에 제가 못찾는건지 안보여서 알고 계신내용 있으신가요?

음 이전 레거시 스터디 목록을 누르면 볼 수 있어야 한다고 하던데 그게 다른 회원도 포함인지는 저도 모르겠습니다

기획분들께 확인해보겠습니다!

@05AM
Copy link
Collaborator Author

05AM commented Mar 21, 2024

근데 저희 스터디 상세(홈 화면) 페이지 접근시 스터디 멤버가 아닌 회원도 해당 페이지에 접근 가능한가요?
기능 명세서에 제가 못찾는건지 안보여서 알고 계신내용 있으신가요?

음 이전 레거시 스터디 목록을 누르면 볼 수 있어야 한다고 하던데 그게 다른 회원도 포함인지는 저도 모르겠습니다

기획분들께 확인해보겠습니다!

피그마 확인해보니 내정보에만 레거시 활동 기록이 있고 다른 팀원에게 이전 기록이 보이는 뷰는 없는 것 같습니다.
그래서 팀원들에게만 스터디 내용이 보이는게 맞지 않을까 싶습니다!

이후에 검색기능이 생긴다면 변동이 있을것 같지만 스터디원인지 검사하는 로직을 추가하는게 좋을까요?

@zxcv9203
Copy link
Collaborator

피그마 확인해보니 내정보에만 레거시 활동 기록이 있고 다른 팀원에게 이전 기록이 보이는 뷰는 없는 것 같습니다. 그래서 팀원들에게만 스터디 내용이 보이는게 맞지 않을까 싶습니다!

이후에 검색기능이 생긴다면 변동이 있을것 같지만 스터디원인지 검사하는 로직을 추가하는게 좋을까요?

이게 정책에 따라 달라질 부분 같은데 검색이 존재하더라도 상세 페이지는 접근하지 못하게 할 수 있을거라 생각해서 (기존 스터디가 어떻게 진행하고 있는지 못보게 하기 위함) 기획에게 질문을 해야하는 부분인 것 같습니다.

만약 제3자는 신청만 가능하고 스터디 내부 정보를 볼 수 없게 막는 형태라면 로직을 추가해야 할 것 같고 제 3자도 자유롭게 다른 스터디 정보를 볼 수 있다면 추가하지 않아도 좋을 것 같습니다.

단, 제 3자가 어디까지 볼 수 있느냐에 대한 정의가 필요할 것 같습니다..!

@05AM
Copy link
Collaborator Author

05AM commented Mar 23, 2024

@zxcv9203

이후에 검색기능이 생긴다면 변동이 있을것 같지만 스터디원인지 검사하는 로직을 추가하는게 좋을까요?

해당 부분에 관한 논의를 생성했습니다
논의에서 답장해주시면 감사하겠습니다!

discussion: #107

@05AM
Copy link
Collaborator Author

05AM commented Mar 26, 2024

이게 정책에 따라 달라질 부분 같은데 검색이 존재하더라도 상세 페이지는 접근하지 못하게 할 수 있을거라 생각해서 (기존 스터디가 어떻게 진행하고 있는지 못보게 하기 위함) 기획에게 질문을 해야하는 부분인 것 같습니다.

만약 제3자는 신청만 가능하고 스터디 내부 정보를 볼 수 없게 막는 형태라면 로직을 추가해야 할 것 같고 제 3자도 자유롭게 다른 스터디 정보를 볼 수 있다면 추가하지 않아도 좋을 것 같습니다.

단, 제 3자가 어디까지 볼 수 있느냐에 대한 정의가 필요할 것 같습니다..!

기획팀께 문의 드리니 스터디의 상세 페이지는 모든 유저가 볼 수 있는 것 같습니다. 따라서 해당 부분은 후에 요구사항이 변경되면 반영하도록 하겠습니다!

@05AM 05AM merged commit c7c69e4 into dev Mar 26, 2024
1 check passed
@05AM 05AM deleted the STMT-62-api_test_get_study branch April 3, 2024 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-0 바로 리뷰가 필요해요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants