traPブログについての説明を追記 #279
traPブログについての説明を追記 #279
Conversation
| label="リンク" | ||
| required | ||
| :class="$style.labeledForm" | ||
| > | ||
| <form-input | ||
| v-model="formValues.url" |
There was a problem hiding this comment.
このときvalidationがおかしくなっていそうです
https://{ なんか }
を入れると「追加」ができるようになる感じです
そもそも入力欄を消しちゃってもいいのでは…？
という気がします
There was a problem hiding this comment.
よく分からなかったのですが、各フィールドに何を入れたときにどういうバリデーションエラーになってほしいってことですか？
そもそも入力欄を消しちゃってもいいのでは
の入力欄ってサービス名に何を選んだときのどれのことですか？
There was a problem hiding this comment.
Blogを追加する際のURL validationについて言及してました
ここで僕が気になっているのは
- URL欄を作る必要がないのではないか
- フロントエンド側ですでにUserID情報は持っているので、改めて入力させるとミスタイプの可能性もあります
- もしURLを作るにしても、validationがおかしいのではないか
https://foohttps://hogeでも通ってしまうので、欄を用意するならhttps://trap.jp/author/{ なにか }でvalidationしたほうがよさそう
ってここまで書いていて思ったんですが、このBlogって個人ブログのことを指してますか？
それだとしてもHomePageと若干競合してそうだったのでtrapのものだと勝手に思ってしまってました
There was a problem hiding this comment.
HomePageはポートフォリオとかで、Blogはtrap.jpではなくて個人ブログのこと(trap.jpに限らないブログだったけどtrap.jpを別で用意するようにしたのでtrap.jp以外のブログ)を想定してると思います
There was a problem hiding this comment.
ポートフォリオって言ってブログのリンク貼ってる人とかポートフォリオとブログが合体してる人とかもいるので確かにBlogはいらないような気もしてきました
別で持ってる人のことも考えると、HomePageを複数設定できるようにしておけばいいですかね
There was a problem hiding this comment.
個人的にはそのような実装のほうがいいと思います
trap.jpのブログに関しては選択とかは必要なくって確定でユーザーページに表示するって感じの仕様になってましたよね確か
There was a problem hiding this comment.
OKです、そうします(多分どこかのPRでHomePageが複数選択できちゃうバグ直してるのですが、そっちにも影響するので見かけたらちょっと待ってもらえると助かります)
そうですね、traPのIDに紐づいて既に公開されているものなので、全員設定の余地なく公開にしています
|
@mehm8128 |
|
多分マージせずにcloseでいいかなーってなってます |
|
承知しました！ |
close #273
Blogを選択したときだけ表示でもいいような気もする