Skip to content
This repository has been archived by the owner on Jul 8, 2020. It is now read-only.

[help]Fix #109 #113

Closed
wants to merge 10 commits into from
Closed

[help]Fix #109 #113

wants to merge 10 commits into from

Conversation

lv7777
Copy link

@lv7777 lv7777 commented Jun 18, 2016

概要

新しくtest関数を作り、その中でidling状態のときをsubscribeして履歴を追加する変数と設計を作りました。
一応できましたが、 #109の内容はhistory classを作るということなので変更と合っていません。
そこでヘルプを貰いたいです。

詳細

まず、

  • 保存機能を実現する実態はどこに作るのか?

もし、framecontrollerと分けるならその理由。同じにするならどうやってhistoryclassから呼び出すか?

  • 具体的なhistory classの内容
  • そのclassはどこで使われる予定なのか?

等々の指示がもらえるとありがたいです。
結構デカいプロジェクトで、どこがどうつながっているのか把握するのに手間取りました・・・(特にframescontroller.framesがどんな感じで更新されていくかとか・・・)

お願いします。


This change is Reviewable

@shundroid shundroid mentioned this pull request Jun 19, 2016
@shundroid
Copy link
Member

@lv7777 PRありがとうございます。
マージ先のブランチを master -> develop に変更したため、
#114 ←こちらのPRに書きたいと思います。よろしくお願いします。

@shundroid shundroid closed this Jun 19, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants