AzusaYoshinaga / chat-space

0 stars 0 forks source link

【依頼】ChatSpace挙動確認 #23

Open AzusaYoshinaga opened 4 years ago

AzusaYoshinaga commented 4 years ago

IPアドレス

18.180.36.202

yasunamiura commented 4 years ago

吉永さん、ご提出ありがとうございます! 挙動確認させて頂いたところ、以下の不備が見受けられましたので修正をお願いします!

[修正箇所] ①削除ボタンのデザインを模範の通りにしましょう ②追加ボタンを押すと追加されたユーザーがundefinedになっています

ChatSpace

模範

ChatSpaceSample

③サイドバーにあるグループ名と最新メッセージの下線を消しましょう

ChatSpace

④Edit文字色を水色にしましょう ⑤メッセージにCSSを当てましょう

ChatSpace

模範

ChatSpaceSample

⑥CSSを当てましょう

ChatSpace

模範

ChatSpaceSample

⑦CSSを当てましょう

ChatSpace

模範

ChatSpaceSample

修正し次第、以下のChatSpace完了報告フォームに再度こちらのissueのURLを記載していただき提出お願いします! ※新しいissueを作っていただく必要はありません https://docs.google.com/forms/d/e/1FAIpQLSfNKbknajjCthhkGSyIkhspu2hOAxnESHweNLX2LQJdlO5vGw/viewform

AzusaYoshinaga commented 4 years ago

改修

1 削除の赤い枠 2 追加ボタンのundifined

原因: config/ seacrets.yml が消えたため。

解決法: configにseacrets.ymlファイル作成 その中に以下を記述

development: secret_key_base: ~ aws_access_key_id: <%= ENV["AWS_ACCESS_KEY_ID"] %> aws_secret_access_key: <%= ENV["AWS_SECRET_ACCESS_KEY"] %>

test: secret_key_base: ~

production: secret_key_base: <%= ENV["SECRET_KEY_BASE"] %> aws_access_key_id: <%= ENV["AWS_ACCESS_KEY_ID"] %> aws_secret_access_key: <%= ENV["AWS_SECRET_ACCESS_KEY"] %>

Screenshot from Gyazo

Screenshot from Gyazo

Screenshot from Gyazo

Screenshot from Gyazo

3 以降は電波が切れそうなので場所を移動して送ります。

AzusaYoshinaga commented 4 years ago

#改修 Screenshot from Gyazo

AzusaYoshinaga commented 4 years ago

改修 ③サイドバーにあるグループ名と最新メッセージの下線を消しましょう

a{ text-decoration: none; } 使用

Screenshot from Gyazo

.groups{ background-color: #2f3e51; height: calc(100vh - 100px); overflow: scroll; .group{ box-sizing: border-box; padding: 20px 20px 0; a{ text-decoration: none; } &name{ color: $white; font-size: $side_group_name;
} &
message{ color: $white; font-size: $side_group_message; margin-top: 5px; } } }

AzusaYoshinaga commented 4 years ago

#改修 ④Edit文字色を水色にしましょう

「colorを水色に変更」 &__button { color: skyblue; text-decoration: none;

「コード」 .right-header{
border: 1px solid $main_blue; color: $main_blue; float: right; height: 40px; line-height: 40px; margin-top: 28px; padding: 0 20px; &__button { color: skyblue; text-decoration: none; } }
}

Screenshot from Gyazo

AzusaYoshinaga commented 4 years ago

⑤メッセージにCSSを当てましょう

_message.htm.lhamlと_chat-maine.scssで .apper_messageが齟齬していたためエラーを出したり齟齬が出て名前と日付を横並びにできなかった。

Screenshot from Gyazo

AzusaYoshinaga commented 4 years ago

⑥CSSを当てましょう

/new.html.haml = render "devis/shared/links" ここからchatspace-matelials/devis/sharred/_link.html.hamlへ

ビューファイルの素材のダウンロードができていなかったので設定。

Screenshot from Gyazo

Screenshot from Gyazo

Screenshot from Gyazo

AzusaYoshinaga commented 4 years ago

⑦CSSを当てましょう

検証してacount-page__inner inner__left acount-pageheader

VScordに上記を入力して検索 app/view/devise/registration/new.html.hamlから stylesheet/_users.scssにaタグを記入して下線を消す

a{ text-decoration: none; }

下線が消える

Screenshot from Gyazo

Screenshot from Gyazo

AzusaYoshinaga commented 4 years ago

結果 Screenshot from Gyazo

mo1015 commented 4 years ago

吉永さん再提出ありがとうございます! 前回修正依頼した箇所の修正が確認できません。 本番環境で実装できているか再度確認をお願いします!

確認でき次第、以下のChatSpace完了報告フォームに再度こちらのissueのURLを記載していただき提出お願いします! ※新しいissueを作っていただく必要はありません https://docs.google.com/forms/d/e/1FAIpQLSfNKbknajjCthhkGSyIkhspu2hOAxnESHweNLX2LQJdlO5vGw/viewform

miyu-yamamoto commented 4 years ago

修正と再提出ありがとうございます! かなり完成に近づいてきましたね、素晴らしいです✨ 今回新たに修正をお願いしたい箇所があるので、あと少し頑張りましょう。

○ サイドバーに表示されているグループごとの間隔が狭く少々見辛くなってしまっているので  見本を参考にして間隔を空けましょう

スクリーンショット 2020-04-03 16 06 17

↓こちらが見本です

スクリーンショット 2020-04-03 16 13 20

○ グループ編集画面にてグループを作成したメンバーにも「削除」が表示されてしまうので、表示されないようにしましょう

スクリーンショット 2020-04-03 16 07 36

確認でき次第、以下のChatSpace完了報告フォームに再度こちらのissueのURLを記載していただき提出お願いします! ※新しいissueを作っていただく必要はありません https://docs.google.com/forms/d/e/1FAIpQLSfNKbknajjCthhkGSyIkhspu2hOAxnESHweNLX2LQJdlO5vGw/viewform

yasunamiura commented 4 years ago

再提出ありがとうございます! 挙動確認させて頂いたところ、前回と同じ不備が見受けられましたので修正をお願いします!

※再度デプロイしていただき、本番環境に反映されているかどうか確認をお願いします!

修正し次第、以下のChatSpace完了報告フォームに再度こちらのissueのURLを記載していただき提出お願いします! ※新しいissueを作っていただく必要はありません https://docs.google.com/forms/d/e/1FAIpQLSfNKbknajjCthhkGSyIkhspu2hOAxnESHweNLX2LQJdlO5vGw/viewform

yasunamiura commented 4 years ago

再提出ありがとうございます! 挙動確認させて頂いたところ、間隔の修正は確認できました。素晴らしいです! 前回の不備が1件と新たな不備が見受けられましたので修正をお願いします!

[修正箇所] 【前回と同じです】 ○グループ編集画面にてグループを作成したユーザー(current_user)にも「削除」が表示されてしまうので、表示されないようにしましょう

ChatSpace

【新しい不備】 ◯サイドバーの文字の縦列を揃えましょう

ChatSpace

修正し次第、以下のChatSpace完了報告フォームに再度こちらのissueのURLを記載していただき提出お願いします! ※新しいissueを作っていただく必要はありません https://docs.google.com/forms/d/e/1FAIpQLSfNKbknajjCthhkGSyIkhspu2hOAxnESHweNLX2LQJdlO5vGw/viewform

AzusaYoshinaga commented 4 years ago

#チャットグループ編集の改修 Screenshot from Gyazo

Screenshot from Gyazo

AzusaYoshinaga commented 4 years ago

33 チャットグループ編集の改修

コード:  書き加えて反映させた

Screenshot from Gyazo

chiii220 commented 4 years ago

吉永さん、ご提出ありがとうございます! 修正箇所確認できました。素晴らしいです! あと数点、以下の修正が見受けられましたので修正をお願いします! もう少しです!!

[修正箇所] ①サイドバー 一番上のグループ名の上に「20px」の余白をいれる

模範

サイドバー見本

現在

スクリーンショット 2020-04-04 10 41 56

②インクリメンタルサーチ後、Saveボタンを押すと「更新」の文字が出る (ご自身で確認されて見てください) 該当箇所

スクリーンショット 2020-04-04 10 47 41

※カリキュラムの中にある、模範のchatspaceと仕様書を参考に、修正をお願いします。 [模範のchatpsace] https://chat-space-sample.herokuapp.com/ [仕様書] https://tech-master.s3.amazonaws.com/uploads/curriculums//03e925297f3843c603f028b177b4e86b.png

修正し次第、以下のChatSpace完了報告フォームに再度こちらのissueのURLを記載していただき提出お願いします! ※新しいissueを作っていただく必要はありません https://docs.google.com/forms/d/e/1FAIpQLSfNKbknajjCthhkGSyIkhspu2hOAxnESHweNLX2LQJdlO5vGw/viewform

AzusaYoshinaga commented 4 years ago

[修正箇所] ①サイドバー 一番上のグループ名の上に「20px」の余白をいれる

Screenshot from Gyazo

Screenshot from Gyazo

chiii220 commented 4 years ago

吉永さん、ご提出ありがとうございます! 確認させて頂いたところ、全く問題ございませんでしたのでLGTMとさせて頂きます🎉🎉🎉 おめでとうございます! お疲れ様でした^^

AzusaYoshinaga commented 4 years ago

まだ、できていません。 グループ編集で追加されたメンバーが、追加ボタンで再度追加できるように名前が重複しています。追加したら同じ名前がメンバーとし反映します。DBで確認しましたら。それらの名前は同一人物です。それのエラーを見つけました。

そして、Saveを押すと更新されると反映されるところを直していません。

chiii220 commented 4 years ago

ご連絡ありがとうございます。

大変申し訳ございません。 こちらで確認したところ、更新ボタンに関しては必須箇所ではありませんでしたのでLGTMを出させて頂きました!

また、 追加されたメンバーが追加ボタンで重複している、に関しては、 仕様上、表示されるようになっております。 吉永さんの場合、追加ボタンを押してもグループメンバーは増加はしないので問題ございません。

更新ボタンに関してはエラー箇所の特定までされているとのことですので、 修正は可能だと思いますが、今後の提出フォームは不要でございます。

AzusaYoshinaga commented 4 years ago

#質問 まだ、できていません。 Saveを押すと更新されると反映されると指摘されていたところを直していません。

グループ編集で追加されたメンバーが、追加ボタンで再度追加できるように名前が重複しています。追加したら同じ名前がメンバーとし反映します。DBで確認しましたら。それらの名前は同一人物です。それのエラーを見つけました。

まだ、chatspaceは終わっていませんのでお手数ですが確認お願いします。

chiii220 commented 4 years ago

吉永さん、ご連絡ありがとうございます! もちろん確認させて頂きます! 修正お待ちしていますね。

AzusaYoshinaga commented 4 years ago

グループ編集のページでメンバを追加し、「Save」ボタンをクリックすると一瞬「更新する」と変化する。(edit)

新規登録画面では、新しいメンバーを追加するときに「Save」ボタンをクリックすると一瞬「登録する」と変化する。(new)

調査: 検索する、Saveボタンをクリックする。 仮説:    1  data-disable-withが怪しい。       data-disable-withはhamlだが、HTML(検索時、RailsのView)       コンパイル作業をする。

   2  根幹設定をしているのは、大体がConfig/application.rb       Config/application.rbファイル:        require 'rails/all'

    (ファイル内)

require_relative 'boot' require 'rails/all'

 # Require the gems listed in Gemfile, including any gems  # you've limited to :test, :development, or :production. Bundler.require(*Rails.groups)

module ChatSpace class Application < Rails::Application config.generators do |g| g.stylesheets false g.javascripts false g.helper false      (←ここをみる) g.test_framework false end config.action_view.automatically_disable_submit_tag = false config.i18n.default_locale = :ja config.time_zone = 'Tokyo' end end

   3  edit「更新する」、newは「登録する」となるので、この二つは連動する。

作業: data-disable-withを消去する。 消去してみると「Save」の文字列が消える。

JaveScript上で何かあるか探してみる。(例: users.js, 〜js)

原因: config/ local/ !ja.yml

helpers: select: prompt: 選択してください submit: create: Save submit: 保存する update: Save

create: 更新する   となっていたので updete: Save に変更する。

create: Save subumit: 保存する update: Save

EditとNewのページでのSaveボタンの挙動は連動しているので、ここでコードを正す。

chiii220 commented 4 years ago

吉永さん、 今確認致しましたところ、Saveボタン機能の修正が出来ておりました! LGTMです🎉🎉🎉 お疲れ様でした!!

AzusaYoshinaga commented 4 years ago

グループ編集のページでメンバを追加し、「Save」ボタンをクリックすると一瞬「更新する」と変化する。(edit)

新規登録画面では、新しいメンバーを追加するときに「Save」ボタンをクリックすると一瞬「登録する」と変化する。(new)

調査: 検索する、Saveボタンをクリックする。 仮説:    1  data-disable-withが怪しい。       data-disable-withはhamlだが、HTML(検索時、RailsのView)       コンパイル作業をする。

   2  根幹設定をしているのは、大体がConfig/application.rb       Config/application.rbファイル:        require 'rails/all'

    (ファイル内)

require_relative 'boot' require 'rails/all'

 # Require the gems listed in Gemfile, including any gems  # you've limited to :test, :development, or :production. Bundler.require(*Rails.groups)

module ChatSpace class Application < Rails::Application config.generators do |g| g.stylesheets false g.javascripts false g.helper false      (←ここをみる) g.test_framework false end config.action_view.automatically_disable_submit_tag = false config.i18n.default_locale = :ja config.time_zone = 'Tokyo' end end

   3  edit「更新する」、newは「登録する」となるので、この二つは連動する。

作業: data-disable-withを消去する。 消去してみると「Save」の文字列が消える。

JaveScript上で何かあるか探してみる。(例: users.js, 〜js)

原因: config/ local/ !ja.yml

helpers: select: prompt: 選択してください submit: create: Save submit: 保存する update: Save

create: 更新する   となっていたので updete: Save に変更する。

create: Save subumit: 保存する update: Save

EditとNewのページでのSaveボタンの挙動は連動しているので、ここでコードを正す。

Screenshot from Gyazo

AzusaYoshinaga commented 4 years ago

エラー修正

既存のuserの名前が出る

インクルメントサーチをした時、グループ編集で既に追加されているチャットメンバーの名前が新たに追加ボタンと共に表示されている。(その人物は、DBで同一人物と確認済み)

コードを改修 作業:users.contller

変更 def index @users = User.where('name LIKE(?)', "%#{params[:name]}%").where.not(id: current_user.id). respond_to do |format| format.html format.json end end

変更後: @users = User.where('name LIKE(?)', "%#{params[:name]}%").where.not(id: current_user.id).where.not(id: params[:user_ids])

.where.not(id: params[:user_ids])を追加することで既に追加しているuserは追加しない。 となる。

よって、既存のuserの名前が新たに追加ボタンと共に表示されなくなった。

Screenshot from Gyazo

chiii220 commented 4 years ago

吉永さん、提出ありがとうございます! 今確認致しましたところ、Saveボタン機能の修正が出来ておりました! LGTMです🎉🎉🎉