すべてのユーザーが表示されないバグの修正 #9

Merged
merged 1 commit into from Jan 19, 2017

Projects

None yet

2 participants

@hatappi
Contributor
hatappi commented Jan 19, 2017

検証URL

https://connpass.com/event/47492

バグ内容

画像が設定されていないユーザーが複数人いる場合に2人目が表示されない

原因

compassではfacebookやtwitter, githubから連携され各サービスで設定されているアバターを使用すると思います。
しかしこれらを利用していない場合に https://connpass.com/static/img/common/user_no_image.gif の画像が使われるようです。

今回修正した部分にてavatarのurlを指定 indexOf でその時のloopのindexとチェックしていたため同じ画像の場合indexが異なるためユーザーとしてピックアップされていなかったようです

今回に関してユーザー情報としてとってきていたnameとavatarを組み合わせてチェックするようにしてみました

@hatappi
Contributor
hatappi commented Jan 19, 2017 edited

@yoshiko-pg
バグらしきものの修正をしてみました
ご確認よろしくお願いいたします 🙇

@yoshiko-pg yoshiko-pg merged commit 384e2e8 into yoshiko-pg:master Jan 19, 2017
@yoshiko-pg
Owner

@hatappi 気付いてませんでした!!ありがとうございます 😂
LGTM!!!

eeab3dc2-582c-8697-2520-3acf8fd05baf

@yoshiko-pg
Owner

github-pages反映しました!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment