tecyokomichi / WebAppLearning

0 stars 0 forks source link

学習/横道/従来型Webアプリ記録用PR #1

Closed tecyokomichi closed 5 years ago

tecyokomichi commented 6 years ago

https://github.com/tecowl/techno-people/issues/109
従来型Webアプリ学習記録のためのPRです

tecyokomichi commented 6 years ago

画面キャプチャ

2018-09-10 18 27 32

2018-09-10 18 30 28

2018-09-10 18 29 42

2018-09-10 18 32 11

2018-09-10 18 31 20

2018-09-10 18 28 36

2018-09-10 18 33 33

2018-09-10 18 34 18

2018-09-10 18 35 12

akm commented 6 years ago

@tecyokomichi 画面がどのように遷移して、どこでどう処理が動くのかよくわからないので、画面遷移図を作ってもらえますか? 作者関係の画面遷移については書いてみたので、残りのボタンやリンクについて記述をお願いします。 遷移の際のメソッドとパスが書いてあるとなお良いです。

https://cacoo.com/diagrams/xGYVMTizzIWO2uWc/74C08 を変更しちゃって構いませんのでよろしくおねがいします。

tecyokomichi commented 6 years ago

@akm 画面遷移図の件について承知しました

tecyokomichi commented 6 years ago

@akm 画面遷移図を整理しました

untitled

https://cacoo.com/diagrams/xGYVMTizzIWO2uWc/74C08

akm commented 6 years ago

@tecyokomichi 直してほしいということではなく、こういう要望があったら簡単に対応できそうですか?画面遷移図の修正なども含めて。

akm commented 6 years ago

@tecyokomichi 僕も指摘から抜けてましたが、自動テストの実装は難しいでしょうか?

tecyokomichi commented 6 years ago

@akm https://github.com/tecyokomichi/WebAppLearning/pull/1#issuecomment-422600650 それほど難しくなく対応できると思います

https://github.com/tecyokomichi/WebAppLearning/pull/1#issuecomment-422600876 これは難しいです

akm commented 6 years ago

@tecyokomichi 昨日話したとおり、Webサーバ側の方はキリの良いところでやめましょう。 心残りがあったらIssueとして残しておいて、再度Webサーバを作る際の学習時にやり直すと良いと思います。

tecyokomichi commented 6 years ago

@akm main.js の長すぎるメソッドをどうにかしたところで一区切りにしたいと考えます