search

kyoshida-aim / RailsTraining

Ruby on Rails研修のPR作成・進捗管理用
https://ky-railstraining-2019.herokuapp.com/
1 stars 0 forks source link

ステップ24-1: タスクにラベルをつけられるようにしよう #45

Open kyoshida-aim opened 5 years ago

kyoshida-aim commented 5 years ago

概要

理由

確認方法

ラベル一覧 image

ラベル更新 image

やっていないこと

ラベルの詳細ページの実装 ラベル自体の設定項目は名前のみなのでわざわざ詳細ページを作るほどじゃないと思います。

相談事項

ラベルのCRUD作成だけで結構な作業量になりそうなのでPRは分けようかと思います PR1(このPR) ラベルのCRUD実装 PR2(予定) タスクにラベルをつけられるようにする PR3(予定) ラベルでタスクを検索できるようにする

kyoshida-aim commented 5 years ago

ステップ24はPRを3段階分割する予定。 このPR内でラベルの基本的な実装は終わったのでWIPを外しました。 次のPRでタスクとの関連付けを行います

kyoshida-aim commented 5 years ago

ちなみにLabelが増えたことで、管理ツールの追加編集の予定はあります？

タスクにラベルを設定できるように変更した際に、タスク詳細ページにはラベルを表示するようにします(一覧画面に表示しようとするとレイアウトが崩れそうなので現時点では予定はないです) タスク詳細ページの変更に合わせて管理画面のタスク詳細でもラベルを表示する予定はあります。

kyoshida-aim commented 5 years ago

20d506e presenceのテストにlengthのテストが混じってるので分けてpushし直します

kyoshida-aim commented 5 years ago

この後コンフリクトを解消します。

kyoshida-aim commented 5 years ago

マージ解消コミット作成

kyoshida-aim commented 5 years ago

(一覧画面に表示しようとするとレイアウトが崩れそうなので現時点では予定はないです)

一覧画面に表示しないと不自然な気がしてきたので実装します(このPRではやりません)

kyoshida-aim commented 5 years ago

ラベルの登録数に制限設けておらず...対応します

kyoshida-aim commented 5 years ago

ラベル作成時にユーザー側にバリデーションを走らせようとしたのですが、上手く関連づけることができず断念。

holysugar commented 5 years ago

:+1: マージについては他の見てから考える