Merged
Conversation
サイトのサポートをするURLがある場合に記述できるようにしました。 このURLは以下の2か所で表示されます。 ・管理者メニューのトップ ・ログイン後のヘッダーメニュー また、サポートページがパスワード付きページに設定されている場合、パスワードも表示するようにしています。 サポートページの設定は.envで行います。 .env.exampleに書き方として以下を追加しました。 # support info (If there is a support URL and if a viewing password is required.) #INDIVIDUAL_SUPPORT_URL= #INDIVIDUAL_SUPPORT_PASSWORD=
masaton0216
reviewed
Sep 19, 2024
Contributor
masaton0216
left a comment
masaton0216
left a comment
There was a problem hiding this comment.
２件コメントしました。
※「なるはや」とのことなので状況次第で今回はスルーしてマージしてしまってもいいとは思います。
- 他用途への転用可能性
- 個別URL表示の依存度
| <a class="nav-link dropdown-toggle" href="#" id="dropdown_support" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">サポート情報</a> | ||
| <div class="dropdown-menu dropdown-menu-right" aria-labelledby="dropdown_auth"> | ||
| <a class="dropdown-item" href="{{config('connect.common_support_url')}}" target="_blank">共通サポートページ <i class="fas fa-external-link-alt"></i></a> | ||
| @if (config('connect.individual_support_password')) |
Contributor
There was a problem hiding this comment.
- 個別サポートページURLの表示が、パスワードの有無に左右されるのは意図した挙動ですか？（コピペミスではない？）
- パスワード不要の個別サポートページというものもあり得た場合、URL表示がパスワード有無に依存してると十何に対応できないなと思った次第です。これに対応する場合、ifの条件はindividual_support_urlの有無でもいいのかもしれないなと。
Contributor
Author
There was a problem hiding this comment.
個別サポートページURLの表示が、パスワードの有無に左右されるのは意図した挙動ですか？（コピペミスではない？）
ミスでした。サンキュー。
パスワード不要の個別サポートページというものもあり得た場合、URL表示がパスワード有無に依存してると十何に対応できないなと思った次第です。これに対応する場合、ifの条件はindividual_support_urlの有無でもいいのかもしれないなと。
個別サポートページの表示とページ閲覧パスワードの表示はそれぞれ独立して判定するようにしました。
パスワード不要の個別サポートページもあると思いますね。
| {{-- マイページではサポートメニューは表示しない --}} | ||
| @else | ||
| <li class="nav-item dropdown"> | ||
| <a class="nav-link dropdown-toggle" href="#" id="dropdown_support" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">サポート情報</a> |
Contributor
There was a problem hiding this comment.
機能（表示内容）が「サポート」という業務に強く寄っているように見受けられますが、下記文言もenvから変更できるようにしておけば、柔軟に様々表示できる「外部サイトリンクエリア」として利用できそうに感じたのですが、どうでしょう？
- サポート情報
- 共通サポートページ
- 個別サポートページ
Contributor
Author
There was a problem hiding this comment.
なるほどです。確かに。
考えてみます。
サンキュー。
Contributor
Author
There was a problem hiding this comment.
検討した結果、以下で行ってみます。
- 今回のヘッダーへの追加は「サポート」に寄ったものとして機能追加します。
Connect-CMSはOSSですが、株式会社オープンソース・ワークショップに限らず、ユーザが何らかの契約のもと、サポートを受けてサイト運営することはスムーズな情報共有を実現するためには良いことかなと思っています。
そのため、サポート関係は独立した形で定義できるようにして、サービスを使う側も、提供する側もわかりやすくしたいと思いました。 - 外部リンクなどの情報表示も良いね。と思うので、これはそういう要望が出てきた際に、「サポート」の横に追加する方向で行きます。
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
概要
サイトのサポートをするURLがある場合に記述できるようにしました。
このURLは以下の2か所で表示されます。
・管理者メニューのトップ
・ログイン後のヘッダーメニュー
また、サポートページがパスワード付きページに設定されている場合、パスワードも表示するようにしています。
サポートページの設定は.envで行います。
.env.exampleに書き方として以下を追加しました。
＃support info (If there is a support URL and if a viewing password is required.)
＃INDIVIDUAL_SUPPORT_URL=
＃INDIVIDUAL_SUPPORT_PASSWORD=
レビュー完了希望日
なるはやを希望
DB変更の有無
無し
チェックリスト