Open
Conversation
Author
작업내용
포함내용
기대 효과 및 전략
|
i2na
reviewed
Mar 21, 2026
Member
There was a problem hiding this comment.
임원진 관리에서,
- 활동 연도 대신 기수를 넣으면 좋을 거 같습니다. (
year를cohort로 대체) - studentId를 number로 하면 좋을 거 같습니다. (
20만 저장) - interview는 md까지는 아직 하지말고, text로 받는게 나을거 같습니다. 기존 메인 웹을 보면 한줄소개 그리고 세 질문에 대한 답변이 있는데 이를 싹 없애고, 한줄소개만 조금 더 긴 분량으로 렌더하도록 바꾸면 유지보수가 편해질 거 같습니다.
- 모든 필드의 값을 다 보여줄 거기 때문에 isActive는 없어도 될거 같습니다.
- profileImage에서 이미지 업로드 기능에 대한 구현 계획이 안 보입니다. 이 부분은 일단 간단하게 이미지 url을 넣도록 구현을 하고, 이후에
02-activity-cms.md에서 이미지 업로드 기능이 구현되면 이 부분을 가져와서 동일하게 사용하도록 하면 될 거 같습니다. - 기수별 임원진 목록 필터링은 없어야 합니다. DB에 기수별로 저장하지 않고 지금 당장의 렌더할 기수에 대해서만 저장할 것이기 때문입니다.
Member
There was a problem hiding this comment.
FAQ 관리에서,
- 답변 내용이 길지 않아서, answer도 string으로 놔도 될거 같습니다.
- 모두 렌더를 할거라, isActive는 없어도 될 거 같습니다.
- 굵은 글씨, 링크 삽입 등의 추가 기능은 일단 당장은 안 넣어도 됩니다. 기능 완성 후 시간이 남는다면 넣으면 좋겠습니다.
Member
There was a problem hiding this comment.
대외링크 관리에서,
- iconType은 없어도 됩니다. type 값에 따라 아이콘을 부여하도록 프론트엔드에서 관리하면 됩니다.
- 이것도 마찬가지로, isActive가 불필요합니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.