URLからuser_data消す #4412
Open
URLからuser_data消す #4412
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
公式ドキュメントにannotations, YAML以外からルーティングをロードする方法が紹介されていました。 |
genzouw
reviewed
Mar 4, 2020
| @@ -1,7 +1,7 @@ | |||
| parameters: | |||
| # EC-CUBE default env parameters | |||
| env(ECCUBE_ADMIN_ROUTE): 'admin' | |||
| env(ECCUBE_USER_DATA_ROUTE): 'user_data' | |||
| env(ECCUBE_USER_DATA_ROUTE): '' | |||
There was a problem hiding this comment.
デフォルト値を変更した場合、既存のサイトへの影響があるため
こちらは変更しないでおきたいです。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概要(Overview・Refs Issue)
ref: #4411
コンテンツ管理からページ作った時に
user_data消す実装に関する補足(Appendix)
annotationだとroutingのmatchの優先順位が設定できないので、routes.ymlの最後に記述した。
他でmatchしなかったらチェックする感じ。
懸念事項
プラグインがrouteを追加する場合、該当のURLが既にページ管理から作成されているURLと競合した場合に問題起きそう。
/faqとかありそうテスト（Test)
shoppingなど既に存在しているURLは作成不可なのは確認しました。マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目