Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ii: add missing reused check #632

Merged
merged 1 commit into from Jan 27, 2017
Merged

ii: add missing reused check #632

merged 1 commit into from Jan 27, 2017

Conversation

kou
Copy link
Member

@kou kou commented Jan 25, 2017

@daijiro ここってreusedなチェックって必要そうですか!?

@yoku0825
Copy link

@yoku0825
Copy link

@kou
Copy link
Member Author

kou commented Jan 26, 2017

あざっす!
同じterm IDを対象にしていたので関係がありそうな気がしました!

どっちもterm IDが1なのは気になりますが、このデータのケースだとterm ID 1のtermが更新されやすいだけなのかしら。

@kou kou merged commit 938ab9e into master Jan 27, 2017
@kou kou deleted the ii-add-reused-check branch January 27, 2017 06:54
@kou
Copy link
Member Author

kou commented Jan 27, 2017

問題は解決していないけどこのチェックは必要なのでマージする。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants