added chat_histories for resending messages

[angdev]

40

[angdev]

채팅 코드 저렇게 모듈별로 묶어서 고쳐보려고 하는데 괜찮을까요?

[TintypeMolly]

상관 있지만 그쪽으로 내가 익숙해지도록 해보지.

[TintypeMolly]

집에가서 확인해보겠음.

[TintypeMolly]

숙제가 넘 많아서 주말에 볼게

[angdev]

네

[TintypeMolly]

문제점

1. /qwer asdf 같은거 쳐서 에러가 돌아왔을 경우에는 히스토리에 저장이 안됨. 현재 유저에게만 보이는 메시지는 .fail() 에서 처리하고 있기 때문에 이점 신경써줘야 될 것 같음.
2. 히스토리에 저장되는 시점이 메시지 발송시점이 아니라 발송완료(200 ok 돌아오는) 시점이라 두개가 서로 다를 수 있다.

[angdev]

그럼 일단 보내는 시점으로 고정하는 것으로 해결하면 되겠군요. 2015년 3월 19일 (목) 오후 8:35, 오진균 notifications@github.com님이 작성:

문제점

1. /qwer asdf 같은거 쳐서 에러가 돌아왔을 경우에는 히스토리에 저장이 안됨. 현재 유저에게만 보이는 메시지는 .fail() 에서 처리하고 있기 때문에 이점 신경써줘야 될 것 같음.
2. 히스토리에 저장되는 시점이 메시지 발송시점이 아니라 발송완료(200 ok 돌아오는) 시점이라 두개가 서로 다를 수 있다.

— Reply to this email directly or view it on GitHub https://github.com/TintypeMolly/Yuzuki/pull/52#issuecomment-83519720.

[TintypeMolly]

내가 짠부분 기존설계의 문제점도 발견했는데

1. 화면에 보일 <li></li> 메시지 개체를 만들어내는 함수가 따로 필요할 것 같다. 에러메시지 만들어내는 로직이랑 그냥 일반 채팅 만들어내는 로직이랑 비슷한데 두번 작성되어 있음.
2. 채팅 메시지의 실패시에 무조건 해당 내용으로 메시지를 만들어내게 되어있는데, 이부분이 내가 의도해서 에러 채팅메시지로 써준 부분이 아니라도 채팅으로써 출력해버림. 그래서 채팅창에 거대한 이리야쨩이 등장하지.

이건 따로 이슈로 뺄테니까 이 풀리퀘에서 신경쓰진 마셈.

다만 1번내용은 같이 수정해서 보내면 감사히 받겠다.

[angdev]

2번 이슈 만들어서 저 주세요. 처리하겠음다.

[TintypeMolly]

아냐 내가 할게.

[angdev]

system error 의 경우에도 date 표시가 필요한가요?

[TintypeMolly]

서버에서 date 찍어서 보내줄게.

[TintypeMolly]

일단 사람이 진짜 보낸 챗을 기준으로 함수를 만들어 줘. 서버쪽에서 에러표기도 다 거기에 맞추겠음.

[angdev]

갈아엎는 중인데 지금 귀찮아서 일단 각 컴포넌트 별로 코드만 붙여놨습니다. 코드 정리는 덜 됐어요.

[TintypeMolly]

이거 왠지 prompted_error를 없애는 종착지에 다다를 것 같군. 지금 몇가지 테스트중인데 곧 머지 할듯.

[TintypeMolly]

chat history를 만들고 다음 커밋에 지우다니. 이거 커밋 두개 합쳐서 머지하겠다.

[TintypeMolly]

merged 27c87de45fa87527439782baa651d59268302ee0