riyenas0925 / Sejong_Track_Management

:school: 세종대학교 트랙관리 프로그램
Apache License 2.0
6 stars 2 forks source link

[ENH,CLN] 로그인 실패시 경고문구, 로그인 시 정보는 세션에 저장하여 활용 #180

Closed kimhanui closed 4 years ago

kimhanui commented 4 years ago

여기 말고 커밋메시지로 확인해주삼..

오타:

  • 의 형태도 다 가능. 근데 jstl의 형태가 심플하므로 특별한 경우아니면 jstl썼으면 좋겠음.

이부분 jstl아니고 el

약초캐시는 분께

  1. 이번에 만들면서 toast말고 alert 썻는데(이유: issue #162) 둘 중 하나로 통일 부탁...!
  2. 로그인 실패시 경고문구는 로그인화면에서 패스워드 치는 부분 바로 밑에 만들엇음. 확인 한번만 해주삼!

라고 전해주세요 @2kyung19님^^

riyenas0925 commented 4 years ago

@kimhanui 지금 잠깐 확인했는데 univ, major String 으로 쓴건 잘했네 굳이 연관관계를 먖을 필요가 없어보이긴 하더라고, major 같은경우는 학과를 말하는거지? Jpa에서 department로 하려고 했는데 major가 더 맞는건가?? 구글 번역기 돌렸는데 department 나오길래 ㅋㅋ

추가로 트랙 파일 업로드 할때 자동으로 UserModel 정보 가져와서 학과나 전공 자동으로 선택하는 기능이 더 괜찮으려나?

riyenas0925 commented 4 years ago

@kimhanui 앞으로 진행할 내용있나? 없으면 전애 말한 uis로그인으로 인증하는 기능 진행 허실??

kimhanui commented 4 years ago

@riyenas0925 연관관계 안 맺은건 먼저 회원가입 할 때 <select>이런거 써서 univ id줘야할 것 같아서 일단 string으로 우선 만들엇음. 굳이 안맺어도되면 좋지ㅋ

major는 전공이라는거임. department는 부서나 단과대학느낌이 강해서 major씀

riyenas0925 commented 4 years ago

@kimhanui 회원가입 할때 univ나 major는 셀렉트 박스 같은걸로 만든다음에 title만 String으로 넘기거나 아니면 너가 말한대로 id로 넘기고 findById같은걸로 찾아서 넣어도 될것 같네 ㅇㅇ

riyenas0925 commented 4 years ago

@kimhanui 추가 사항 없으면 어프로브 한다?

kimhanui commented 4 years ago

@kimhanui 앞으로 진행할 내용있나? 없으면 전애 말한 uis로그인으로 인증하는 기능 진행 허실??

  • trackrule은 좀 나중에? @riyenas0925
riyenas0925 commented 4 years ago

@kimhanui trackrule은 trackJudege 부분 하면서 수정이 좀 필요한것 같아서 내가 수정하고 있어서 다른 기능 우선 하고 있으면 될것 같음 ㅇㅇ

kimhanui commented 4 years ago

추가로 트랙 파일 업로드 할때 자동으로 UserModel 정보 가져와서 학과나 전공 자동으로 선택하는 기능이 더 괜찮으려나?

자동으로 초기화되게 하면 편할듯 ㅇㅇ

지금처럼 univ, major를 String 으로 갖고 있는 상태에서는 학과, 전공을 이름 으로 조회하는 메소드 만들어주고 작업하면 편할거같다 @riyenas0925