minhyeok4dev commented 9 years ago

363

테스트하실때 유의할 점이 몇가지 있습니다.

계좌가 등록되어있다고 가정했기 때문에 Account 모델에 미리 설정해둔 레코드가 들어가있어야 합니다. rake db:recreate에 적용해놨으니 이를 돌리시면 됩니다
참가비 개념이 들어가니 TO공지를 생성하실때 반드시 참가비를 지정해주셔야 하고 테스트 회원을 참가 신청 해두셔야합니다.
xls파일을 떨구는 신한은행 기준에 따라 모든 테스트 환경은 MS Excel에서 진행됩니다

테스트를 위한 엑셀 서식- https://www.dropbox.com/s/5w1ud39w8c7wvfp/account_example.xlsx?dl=0

shaynekang commented 9 years ago

음? 날짜가 하루씩 밀려서 들어가는 것 같네요. Local Time이 안 맞는 문제 같은데, 확인이 필요할 것 같습니다. ㅎㅎ

20_bank_account_error

shaynekang commented 9 years ago

수고하셨습니다~

Account는 사용자 계정이라는 표현으로도 쓰기 때문에, 좀 더 구체적인 다른 표현을 사용해야 헷갈리지 않을 것 같습니다. ㅎㅎ 저는 보통 BankAccount라는 표현을 사용했습니다.
음... 입금자명을 Unique하게 만들어주기로 하지 않았나요? 가령 입금자가 '김민혁'이라면 '김민혁-A16D'등으로 만들어서 이 입금자명으로 넣으면 자동으로 납부 처리를 해 주는 식으로 말이죠. ㅎㅎ
납부 여부는 현재 어떻게 체크하고 있나요? 엑셀 업로드할때 이름이 일치하면 공지의 '실시간 TO현황'에서 자동으로 '회비'버튼이 ON 상태로 바뀌는건가요?
축약어가 많네요. 가령 result_arr라던가. 또한 er과 같은 변수명은 보통 error라고 생각하기 마련이죠. ㅎㅎ
테스트가 안 짜여져 있는 것 같습니다. model, lib, service 등은 테스트를 작성해주세요. ㅎㅎ

일단 Parsing은 잘 하는 것 같네요. 다만 실제 라이브서버에 올릴 수 있는 상황은 아닌 것 같습니다. @account = Account.find_by_account_number("110383537755") 같이 다소 하드코딩 된 부분도 있고 말이죠. 수정이 필요한 부분이 많아 보입니다. ㅎㅎ

몇가지 UX적인 개선점도 보이고 말이죠. 막상 봐서는 어떻게 써야하는지, 어떻게 회비가 잘 납부되었다고 확인할 수 있는지 잘 모르겠네요. ㅡ,.ㅡ

일단 코드를 수정하고, 다음 멘토링때 UX적인 부분을 포함해서 다시 한 번 논의해봐야 할 것 같습니다.

여튼 수고하셨습니다~

minhyeok4dev commented 9 years ago

2- 앗.. 저번에 설명드렸었는데 총무와 인터뷰하면서 입금자명 기입에 대한 불편함이 상당히 크다고 결론지어서 다른 방법으로 하기로 했었습니다..!

3- 네 그렇습니다. TO공지를 만들고 참가자 리스트에 해당 회원을 올린다음 회계 업로드를 하면 단원구분별 입금액 기반으로 매칭되는 참가자들에 대해 자동 납부 확인이 적용됩니다

그 외 나머지 부분 피드백 반영했습니다. 실제로 작업하면서 UX적인 부분에 대해 고민이 좀 많았습니다. 기능 우선으로 먼저 구현하여 오늘 인터뷰때 보여주었고, 멘토링 때 내부 회의와 인터뷰 반영해서 최종 결정하면 될것같습니다..!

shaynekang commented 9 years ago

확인했습니다! UX적인 부분 등은 다 같이 살펴보도록 하죠. ㅋ

수고하셨습니다 -_-)/

DonutWorks / Ari

Feature/check payment by account #382

363