プレビュー用にユーザーアイコンを表示する際に、ユーザー名を使用するFeature use user name for get icon #7607
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.
Issue
概要
前提
テキストエリアで
:@username:
と記述するとメンションされたユーザーのアバターが表示される。この時
user_icon_urls
というapiを使用してアバターのアタッチされた全ユーザーのアバター画像URLを取得し、表示するべきユーザーのログインネームで取り出すという処理をしている。本PRではサーバー負荷削減を目的として、
user_icon_url
を使わないよう修正する。変更内容
user_icon_urls
を使わずにアバターを取得できるように変更したuser_icon_urls
関連のファイルとコードを削除した変更確認方法
先に確認いただきたいことの要点をまとめておきます
事前準備(ローカル環境)
feature/use_username_for_get_icon
をローカルに取り込むrails db:reset
するforeman start -f Procfile.dev
でサーバーを立ち上げるローカル環境での確認
komagata
でログインするhttp://localhost:3000/reports/new
にアクセスする:@komagata:
と入力し、プレビュー欄にアバターが表示されることを確認する:@kimura:
と入力し、プレビュー欄にデフォルトのアバターが表示されることを確認する事前準備(ファイルのパス変更)
config/storage.yml
のlocalを以下に書き換えるlib/tasks/bootcamp.rake
の61~64行目を以下に書き換えるlib/bootcamp/setup.rb
の35行目を以下に書き換えるapp/models/user.rb
の605行目を以下に書き換えるreturn
rails db:reset
するファイルパス変更確認
rails bootcamp:oneshot:change_icons_filepath
を実行する事前準備(Google Cloud Storage環境)
config/webpack/development.js
の2行目を削除し、以下を追記するforeman start -f Procfile.dev
でサーバーを立ち上げるGoogle Cloud Storage環境での確認
http://localhost:3000/reports/new
にアクセスする:@komagata:
と入力し、プレビュー欄にアバターが表示されることを確認する:@kimura:
と入力し、プレビュー欄にデフォルトのアバターが表示されることを確認するScreenshot
画面の変更はありません