Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 00:20
    chihiro-adachi edited #4940
  • 00:20
    chihiro-adachi edited #4940
  • 00:20
    chihiro-adachi edited #4940
  • 00:20
    chihiro-adachi edited #4940
  • 00:20
    chihiro-adachi edited #4940
  • 00:20
    chihiro-adachi labeled #4940
  • 00:20
    chihiro-adachi milestoned #4940
  • 00:20
    chihiro-adachi edited #4940
  • 00:20
    chihiro-adachi opened #4940
  • 00:17
    kiy0taka commented #4917
  • 00:17

    kiy0taka on 4.0

    ログイン時に不正な転送先を指定した場合は400を返すよう変更 Merge pull request #4917 from i… (compare)

  • 00:17
    kiy0taka closed #4917
  • Feb 25 19:49
    shyim commented #4902
  • Feb 25 06:00
    kurozumi opened #4939
  • Feb 25 05:03
    okazy commented #4938
  • Feb 25 04:58
    okazy milestoned #4937
  • Feb 25 04:57
    okazy labeled #4937
  • Feb 25 04:57
    okazy edited #4937
  • Feb 25 04:57
    okazy edited #4937
  • Feb 25 04:57
    okazy edited #4937
Yangsin
@Yangsin
プラグインでのログ実装ガイドラインというのも必要そうですが、
本体のログが足りないのも、事実なので実装は進める必要を感じてます。

エラーは、「フィールドグループに追加のフィールドを含んではなりません。」ってやつです

@nobuhiko
この件って、ログとはまた別な課題がありません？

のぶ
@nobuhiko
@Yangsin エラーが起きたときのトレーサビリティ、という部分ですね
ログより、画面とプラグインの問題です
Yangsin
@Yangsin
ですよね。ログ以外でもトレーサビリティあげるべきポイントがあるというふうに理解してます。
のぶ
@nobuhiko
ログというポイントに絞ったとしても、
https://github.com/EC-CUBE/related-product-plugin/
がやたら競合するするいわれる→デバッグする→dump($form->getErrors(true))って書いてようやく原因がわかる
じゃあここに意識しないでもログ出力して置けるようにすれば、（もしくはプラグインの機構直せば）解決するんじゃね？って話の流れのような
Yangsin
@Yangsin
ログで解決しきれるかは難しいところもありそうですが、プラグインがエラーを画面にだせてないなら、プラグインも直さねばという認識はあって、公式のプラグインはちゃんと直していきます。
ログだけで全部解決するのは難しいと思いますが、現状の本番環境でのログの少なさは改善していこうというところですね。
のぶ
@nobuhiko
最終的にそう返されると、「必要な」ログを増やしましょうとしかいいようがないです
Yangsin
@Yangsin
@nobuhiko
うーん、そういうとあれですが、
ポイントは、プラグインが意識しなくてもフォームのエラーログの出力を増やせないか？
ってところですかね？
のぶ
@nobuhiko
@Yangsin いや、フォームに限らずですけど。ポイントプラグインぐらいみんな意識してログを残すのは難しいですよ
Yangsin
@Yangsin
プラグイン側で意識させることが、日に日に増えていってるんじゃね？って話もありそうですね。
Yangsin
@Yangsin
3.0.12 のマイルストーンがついているが、対応が完了していない・完了見込みでないものを 3.0.x のマイルストーンへ移動していきます。
状況の更新もれで、実際には対応完了しているものなどあれば、確認お願いします。
Yangsin
@Yangsin
本日、3.0.12リリースいたしました。ご協力いただきました皆様ありがとうございます。
http://www.ec-cube.net/news/detail.php?news_id=274
引き続き年内で3.0.13を予定しています。
そして、いい加減3.1についてもマイルストーン明確にしていければと思ってますが、そちらについては今しばらくお待ちくださいませ～
Yangsin
@Yangsin
3.0.12のリリースが、めずらしくITmedia マーケヒングに掲載されてます。http://marketing.itmedia.co.jp/mm/articles/1611/01/news083.html
Yangsin
@Yangsin
DockerHubに登録しました。といいつつも、まだリポジトリはありませんが、ちかぢかリポジトリもつくっていく予定。
https://hub.docker.com/u/eccube/
Yangsin
@Yangsin
マイルストーンにかいてるだけだとわかりづらいかなとか意見しづらいかなということで、3.0.13の目安箱Issueをつくってみました。
EC-CUBE/ec-cube#1900
Chihiro Adachi
@chihiro-adachi
価格が高い順/低い順のソートについて、wipでpull request上げてます。
ご意見等あればよろしくお願いいたします。
EC-CUBE/ec-cube#1912
Chihiro Adachi
@chihiro-adachi

#1912 いったん実装完了しています。

最終の仕様もまとめているので、お気づきの点などあればissueまでお願いします。

Yangsin
@Yangsin
EC-CUBE/ec-cube#1912
ready-for-merge にしました。週明けを目処にマージします。
Yangsin
@Yangsin
@/all
EC-CUBE/ec-cube#1570
こちらの機能追加の採用を予定していますが、何か気になる方は今週中めどでコメントお願いします。
Kentaro Ohkouchi
@nanasess
こういう J2EE のパターンってレガシーなんですかね？
http://patterns-wg.fuka.info.waseda.ac.jp/study/14th-yamano.pdf
Yangsin
@Yangsin
プラグインの利用方法レベルで問い合わせも結構あるので、
@shhirose がくれてる以下を3.0.13で、採用しようと思います。
3.1とかでは、Markdownとかいろいろとフォーマットも考えてもいいかなと思いますが、まずは既存のままさっといけそうなので。
EC-CUBE/ec-cube#1645
Yangsin
@Yangsin
先週、 ready-for-merge にしたPullRequestを随時マージしていきます。
Yangsin
@Yangsin
本日 ready-for-merge といっていたものは全てマージしました。
https://github.com/EC-CUBE/ec-cube/pulls?q=is%3Aopen+is%3Apr+label%3A%22Status%3A+ready-for-merge%22
こちらの #1942 #1922 #1910 #1797 #1654 #1645 を12月1日以降でマージしていきます。
Yangsin
@Yangsin
@/all
おくればせながら、AdventCalendar 今年もやりまーす。ぜひご参加を～
http://qiita.com/advent-calendar/2016/ec-cube
Chihiro Adachi
@chihiro-adachi
かきましたー
http://qiita.com/chihiro-adachi/items/50a59d379eb5b1b9d7df
Yangsin
@Yangsin
↑ JetBrainsにレポートしました～サンクス
Yangsin
@Yangsin
JetBrains からOSSサポートをうけれることになりました。Contributorとして数ヶ月アクティブであることと、 ブログなどに利用レビューを投稿していただける場合にライセンスをご利用いただけますので、ご興味ある方はお声がけください。
Kentaro Ohkouchi
@nanasess
PHP7.1 は TaxRuleRepositoryTest で落ちますね...
https://travis-ci.org/nanasess/ec-cube/jobs/180721399
Yangsin
@Yangsin
7.1対応するの結構大変そうな気がしてきました
のぶ
@nobuhiko
3.0.12のパッケージに問題ありそうです？
http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=18318&forum=2&post_id=78961#forumpost78961
Yangsin
@Yangsin
Windowsで解凍するとということなので3.0.12に関わらずパッケージの作成に問題がありそうです
のぶ
@nobuhiko
EC-CUBE/ec-cube#684
Yangsin
@Yangsin
おーまたじっくり確認します
Kiyotaka Oku
@kiy0taka
ギリギリでかきましたー
http://qiita.com/kiy0taka/items/1bd6dae16c8b5c99151a
Yangsin
@Yangsin
3.0.13 ですが、 16日 までのPullRequestをもって、年内でリリースを予定しています。
3.0.14については、1月末から2月上旬を想定しています。
Yangsin
@Yangsin
開発ドキュメントのサイトをリニューアルしました。
http://ec-cube.github.io/
Yangsin
@Yangsin
プラグインのアップデート状況についてまとめてみました。
http://qiita.com/Yangsin/items/cd5eb2416713b0ba2927
Yangsin
@Yangsin
Travisにマージを阻まれている・・・
Kentaro Ohkouchi
@nanasess
Silex2.0 + Symfony3.1 でトップページの表示まではできました。いろいろコメントアウトしてますが。。。
https://github.com/nanasess/ec-cube/tree/improve/silex2
Yangsin
@Yangsin
ドキュメントサイトのプラグインジェネレーターのページ、３．０．１３向けに更新できないでしょうか？
http://ec-cube.github.io/plugin_tutorial-generate
@chihiro-adachi @ryo-endo
こちら3.0.13での開発範囲なので取り込みたく。レビューと確認お願いします。
EC-CUBE/ec-cube#1989
Chihiro Adachi
@chihiro-adachi
@Yangsin 動作確認しました。問題ないかと思います。。
Yangsin
@Yangsin
@chihiro-adachi
一通り対象マージしましたのでパッケージングお願いできますか？（バージョンは試験後にあげたいと思います。）
@kiy0taka
現時点のMasterにたいして Codeceptionまわしてみてもらえますか？Travisから押せばよい？