jidoc01 / RustyDO

A server emulator for Digimon Online v1.5.
GNU Affero General Public License v3.0
53 stars 12 forks source link

flowing message change #2

Closed coolkid8715 closed 2 years ago

coolkid8715 commented 2 years ago

I've replaced the flowing information. Those messages were originally embedded in the game.

jidoc01 commented 2 years ago
  1. 커밋 메시지에서 한글이나 내용과 무관한 메시지는 삭제 바라며, 제목도 커밋 내용을 특정할 수 있게 작성 바랍니다. 또한 기본적인 메시지 작성 방법을 먼저 숙지한 뒤 작성 부탁드립니다: https://gist.github.com/robertpainsi/b632364184e70900af4ab688decf6f53
  2. 원래 게임에 내재되어 있다는 레퍼런스 확인 가능할까요? 제가 아는 것과는 확연히 달라서 그렇습니다.
  3. 현재 해당 메시지들을 config.toml에서 설정 가능하도록 구상 중입니다.
coolkid8715 commented 2 years ago

어.. 죄송합니다, 제가 깃허브를 이번에 거의 처음 써보는지라.. 커밋한것중 가장 마지막이 설명 정리가 되어있습니다.

화면 캡처 2022-08-18 195208 화면 캡처 2022-08-18 195801 화면 캡처 2022-08-18 195815 화면 캡처 2022-08-18 195825

헥스 툴로 뜯어 확인한 디지몬 온라인 메시지입니다. ticker.txt 가 없을때 이렇게 나오더라고요. 헥스 툴은 wxMEdit 라는 헥스툴을 사용했습니다.

jidoc01 commented 2 years ago

확인해보니 클라이언트에 내재되어 디폴트로 된 것은 맞지만, 아카이빙된 클라이언트의 당시 ticker 메시지와 비교해 보건데, 라이브 서버에서 실제로 쓰였는지 여부는 불투명해 보입니다. 초기 클라이언트 설계 시 예외 처리를 위해 개발자가 심어놓았을 가능성도 점쳐집니다. 따라서 당장 수정을 할 이유는 없어 보입니다.

버그 픽스나 비효율적인 구현의 개선, 혹은 issue에서 논의가 진행된 건에 대해서는 곧바로 PR을 주셔도 좋습니다. 이외로 이렇게 별도 논의가 필요한 내용에 대해서는 먼저 issue를 발급을 한 뒤 진행하면 좋을 것 같습니다.

별개로, 커밋 메시지는 아직 guideline을 만족하지 못하고 있습니다. 상기한 링크를 한번 더 정독하시거나 관련한 내용을 좀 더 찾아보시면 좋을 것 같습니다.

감사합니다.

jidoc01 commented 2 years ago

I confirmed that the ticker messages you've put in the code are included in the original client. But it seems not obvious that it was actually used in a live server at that time since they are far different from those in the archived client. So I don't see any reason to fix them.

Closed.