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

[#1] 기능 요구사항 추가.md #2

Merged
merged 7 commits into from
Nov 9, 2024
Merged

[#1] 기능 요구사항 추가.md #2

merged 7 commits into from
Nov 9, 2024

Conversation

ingyeomnote
Copy link
Collaborator

요구사항을 세분화하고, 어떻게 구현할 것인지 생각해보았습니다.
또한, 외부 서비스가 필요한 것은 별도로 표기했습니다.

@ingyeomnote ingyeomnote self-assigned this Oct 29, 2024

Choose a reason for hiding this comment

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

프로젝트 구성에 너무 많은 외부 서비스 의존성이 필요합니다.
컴포넌트 다이어그램으로 구현할 서비스간의 관계와 와 외부 서비스의 의존성등을 시각화해 주세요.

Copy link
Collaborator Author

@ingyeomnote ingyeomnote Nov 1, 2024

Choose a reason for hiding this comment

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

컴포넌트 다이어그램 작성이 처음이라 제대로 작성한 건 지 잘 모르겠습니다ㅠ

@@ -1,7 +1,16 @@
# 컴포넌트 다이어그램
![image](https://github.com/user-attachments/assets/0fcd8db0-9d43-4e60-b9a0-e53b48d59758)
```mermaid

Choose a reason for hiding this comment

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

PlantUML로 컴포넌트 다이어그램을 그려보시는 것을 권장합니다.
docs 디렉토리를 하나 추가하시고 puml 파일로 다이어그램을 추가해주세요.
그리고, 컴포넌트 단위로 분리해서 의존 관계를 설정해주세요.
회원 가입, 로그인, 대여, 반납, 조회 등은 행위입니다.
어떤 컴포넌트가 어떤 행위를 처리할지 설정하고 각 컴포넌트 간의 관계를 시각화해 주세요.

@@ -0,0 +1,27 @@
```plantuml
Copy link

@roam-f-lab roam-f-lab Nov 6, 2024

Choose a reason for hiding this comment

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

PlantUML에서 주석은 ` 문자가 아니라 ' 입니다. 그리고, 이미 파일 확장자에서 puml 이라는 정보가 있으니 주석으로 plantuml 이라는 정보를 추가하지 않아도 괜찮습니다.
추가 피드백은 토요일 세션에서 진행하겠습니다.
이제 기본 적인 뼈대 코드를 구성해주세요.

@ingyeomnote ingyeomnote changed the title 기능 요구사항 추가.md [#1] 기능 요구사항 추가.md Nov 6, 2024
Copy link

sonarcloud bot commented Nov 7, 2024

@ingyeomnote ingyeomnote merged commit c6e6152 into main Nov 9, 2024
2 checks passed
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