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

ページ管理で更新した内容が反映されるよう修正 #126

Merged
merged 2 commits into from
May 8, 2020

Conversation

kurozumi
Copy link
Contributor

@kurozumi kurozumi commented Apr 3, 2020

管理画面のページ管理で更新した内容が反映されないので、
内容が反映されるようテンプレートのパスを修正しました。

管理画面のページ管理で更新した内容が反映されるよう修正しました。
@Yangsin Yangsin added this to the 4.0 milestone Apr 17, 2020
@Yangsin Yangsin added the bug label Apr 17, 2020
@okazy
Copy link
Contributor

okazy commented May 1, 2020

動作確認をさせていただきました。
管理画面からテンプレートを変更可能になっていることを確認しました。

ただ私自身のテンプレートの知識関連が足りておらず、この記述で問題ないかどうかの判断ができていません。
もう少し確認してから取り込みをさせていただきます。

EC-CUBE本体で「プラグインのテンプレートを上書きできるように対応」のプルリクもいただいていて、影響がないかも確認してから取り込みたいと思っています。
EC-CUBE/ec-cube#4527

Copy link
Contributor

@okazy okazy left a comment

Choose a reason for hiding this comment

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

動作とコードを確認しました。
お作法としてはディレクトリ構造も修正した方がよいかと思いますが、互換性確保のためこちらの修正でよいかと思います。

調査結果はIssueにしました。
#128

seeアノテーションコメントを追加しました。

@okazy okazy modified the milestones: 4.0, 4.0.7 May 8, 2020
@okazy okazy merged commit 57836de into EC-CUBE:4.0 May 8, 2020
@okazy
Copy link
Contributor

okazy commented May 8, 2020

ありがとうございます!取り込みました!

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

Successfully merging this pull request may close these issues.

3 participants