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

FileManager:display filePath when you upload a file. #4080

Merged
merged 15 commits into from Feb 13, 2019
Merged

FileManager:display filePath when you upload a file. #4080

merged 15 commits into from Feb 13, 2019

Conversation

ghost
Copy link

@ghost ghost commented Feb 8, 2019

概要(Overview・Refs Issue)

方針(Policy)

  • このPullRequestを作るにあたって考慮したものや除外した内容
    ファイルパス取得時、「//」となってしまうのでTwig側で「/」を削除しています。

実装に関する補足(Appendix)

  • コードだけではわかりづらい点など、実装するにあたってレビューアに追加で伝えておきたいこと
    faviconのPRをマージしています。

テスト(Test)

  • テストを行っている範囲など、レビューアが安心できるような情報
    E2Eテスト作成・実行しています。

相談(Discussion)

  • 相談したいことや意見を求めたいこと
    Windows環境で作ってます。

マイナーバージョン互換性保持のための制限事項チェックリスト

  • マイナーバージョンでは、機能・プラグイン・デザインテンプレート互換性を損なう変更は原則取り込みません。
  • 既存機能の仕様変更
  • フックポイントの呼び出しタイミングの変更
  • フックポイントのパラメータの削除・データ型の変更
  • twigファイルに渡しているパラメータの削除・データ型の変更
  • Serviceクラスの公開関数の、引数の削除・データ型の変更
  • 入出力ファイル(CSVなど)のフォーマット変更

@chihiro-adachi chihiro-adachi added this to the 4.0.2 milestone Feb 12, 2019
@chihiro-adachi chihiro-adachi added the enhancement 機能追加 label Feb 12, 2019
@chihiro-adachi
Copy link
Contributor

@YuYu777
E2Eテストが落ちているのでご確認お願いします。

@nanasess nanasess mentioned this pull request Feb 12, 2019
6 tasks

$('.action-copy').on('click',function(){
updated = $(this).parents('tr').find('.updated');
copy_file_path = $(this).parents('tr').find('.copy-file-path');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nanasess @YuYu777
グローバル変数になってるので修正お願いします

@chihiro-adachi chihiro-adachi added Status: ready-for-merge affected:admin_template 管理画面テンプレートのDOMに影響のある変更 and removed Status: fix-me labels Feb 12, 2019
nanasess added a commit to nanasess/ec-cube that referenced this pull request Feb 13, 2019
FileManager:display filePath when you upload a file.
nanasess added a commit to nanasess/ec-cube that referenced this pull request Feb 13, 2019
FileManager:display filePath when you upload a file.
@nanasess
Copy link
Contributor

messages.en.yaml にも追加が必要ですね

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected:admin_template 管理画面テンプレートのDOMに影響のある変更 enhancement 機能追加 Status: ready-for-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants