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

university의 english_course, accommodation url 삭제 #84

Open
3 tasks
wibaek opened this issue Sep 4, 2024 · 0 comments
Open
3 tasks

university의 english_course, accommodation url 삭제 #84

wibaek opened this issue Sep 4, 2024 · 0 comments
Labels

Comments

@wibaek
Copy link
Contributor

wibaek commented Sep 4, 2024

어떤 기능인가요?

현재 대학의 url 관련 정보는 university_info_for_apply의 정보에서 분리되어 university에 따로 저장되고 있습니다.

  1. 따로 분리한 이유는 이를 프론트에 보내줄 때 링크만 따로 온다면 태그 적용 하기가 쉽기 때문입니다.
  2. university에 따로 저장한 이유는 아마 url 정보는 학기에 따라서 변경되지 않는다고 가정하고 설계했던 것 같습니다.

1번에 관하여, 따로 분리하기보다 프론트 측에서 자동으로 링크를 파싱하여 linkify하는 것이(커뮤니티 포스트와 같이) 더 효율적일 것 같아, 더 이상 필요 없는 작업이고

2번에 관하여 url도 학기마다 변경되는게 있기에 문제가 있습니다.

근본적으로 이제 분리할 필요가 크게 없기에 필드를 삭제함을 건의합니다. 그러나 homepage_url은 대부분 영구적이고 대학의 일반 정보를 담으니 남겨두려 합니다.

질문이나 의견 자유롭게 남겨주세요!

작업 상세 내용

  • (프론트)URL 자동 링크화 적용
  • Entity에서 url 필드 2개 삭제
  • DTO와 서비스 로직 수정

참고할만한 자료(선택)

@wibaek wibaek added the feature label Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant