by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 08:05

    kiy0taka on 20200924

    (compare)

  • 08:02

    kiy0taka on master

    ログのダウンロード機能を追加 注文確認メール(HTML)の見出しが本文と同じフォントサイズに… パッケージのディレクトリ名にタグを追加 and 4 more (compare)

  • 05:46

    kiy0taka on 4.0

    注文確認メール(HTML)の見出しが本文と同じフォントサイズに… Merge pull request #4677 from 9… (compare)

  • 05:46
    kiy0taka closed #4677
  • 05:34
    kiy0taka commented #4677
  • 04:59
    kiy0taka closed #4650
  • 04:59
    kiy0taka commented #4650
  • 04:58

    kiy0taka on 4.0

    ログのダウンロード機能を追加 Merge pull request #4666 from s… (compare)

  • 04:58
    kiy0taka closed #4666
  • 01:32
    nanasess opened #4700
  • Sep 20 02:24
    nanasess commented #4623
  • Sep 20 02:07
    nanasess commented #4697
  • Sep 19 09:07
    izayoi256 synchronize #4697
  • Sep 19 07:27
    izayoi256 synchronize #4697
  • Sep 18 07:18
    nanasess commented #4698
  • Sep 18 06:01
    okazy milestoned #4699
  • Sep 18 06:01
    okazy labeled #4699
  • Sep 18 06:00
    okazy opened #4699
  • Sep 18 04:42
    izayoi256 commented #4697
  • Sep 18 04:34
    okazy milestoned #4698
のぶ
@nobuhiko
@Yangsin それ、プラグイン実装者に左右されすぎじゃ
本体よりプラグインの問題のほうが、ここはある気が
あと、エラーは、「フィールドグループに追加のフィールドを含んではなりません。」ってやつです
Yangsin
@Yangsin
ぐふっ
TSURU
@ttsuru
本来の問題を突き詰めてから実装した方が良さそうですね。
何が問題かが明確になっていなそうな気がしますね
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
かきましたー
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
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にマージを阻まれている・・・