sparcs-kaist / memvers-front

Frontend server for Memvers service
1 stars 0 forks source link

Refactor #4

Closed jwoo0122 closed 5 years ago

jwoo0122 commented 5 years ago
jwoo0122 commented 5 years ago

배포는 npm run build 후에 dist 배포 하시면 됩니다.

jwoo0122 commented 5 years ago

지금 보니까 프론트 앱 제공하는 서버가 이전 코드에 있었군요.. 같은 디렉토리에 다시 살려둘게요.

Sent with GitHawk

jwoo0122 commented 5 years ago

이전처럼 server/main.js 로 만들어 두었습니다. PORT=80 forever start main.js 로 실행할 수 있습니다.

Medowhill commented 5 years ago

PR과 직접적으로 상관있지는 않으나, 현재 API 서버를 고치는 중입니다. 로직은 동일한데, 지금 url이나 응답 field가 정리되지 않은 면이 많아서 좀 고치려고 합니다. 나중에 배포되면 프런트도 그에 맞게 수정해주세요. 로직을 바꿀 필요는 없을거고 url이랑 확인하는 field 정도만 바꿔주시면 될거에요.

jwoo0122 commented 5 years ago
Medowhill commented 5 years ago
2019-03-08 9 49 33

logout을 눌렀을때 다음 팝업이 뜨는걸 발견했습니다. 재현해보려 했는데 안돼서 조건은 모르겠네요. 지금 고칠 필요는 없고 나중에 다시 발견되거나 원인을 찾으면 고쳐도 될 것 같습니다.

jwoo0122 commented 5 years ago

코멘트 해주신 부분들 수정 완료 했습니다. Wheel 기능들은 제가 휠 계정을 가지고 있지 않아서 테스트가 좀 어려운 것 같은데, 휠 테스트가 가능한 계정이 공유된다면 해당 기능 먼저 추가 후 UX / 버그 처리 진행해 보겠습니다.

Medowhill commented 5 years ago

사실 @jwoo0122 님도 휠세미나를 들으셨으니 휠 계정을 사용하는것 자체는 문제가 없을 것 같으니 현 휠장님의 허락을 받고 휠 비밀번호를 알려달라고 해도 될거 같고, 다른 방법으로는 휠 계정에게만 권한을 부여하는 것이 아니라 휠 그룹의 모든 계정에 권한을 부여하는 것이 있는데, 이쪽에 대해서는 현 휠장님과 논의해 보겠습니다.

Medowhill commented 5 years ago

그리고 그와는 별개로 ldap, nugu 디비를 실 서비스용과 분리해서 테스트용 api 서버를 만드는게 여러모로 테스팅에 유리할거 같은데 천천히 준비해보겠습니다.

Medowhill commented 5 years ago

input 초기화랑 위 에러는 별도 이슈로 만들어두고 이 PR은 merge 하겠습니다.