Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GithubのContributionが表示されないバグを修正した #6745

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

yuma-matsui
Copy link
Contributor

@yuma-matsui yuma-matsui commented Jul 20, 2023

Issue

概要

トップページに表示されているGithub Contribution(草)が表示されなくなってしまったのでこれを修正。

image

バグの原因はデータ取得元(Github)の仕様が代わり、Contribution情報の取得ができなくなってしまったため。

変更確認方法

  1. hotfix/fix-github-grassをローカルに取り込む
  2. bin/setup
  3. bin/rails s
  4. hatsunoでログイン
  5. http://localhost:3000に遷移
  6. 画面下部にGithub Contributionのテーブルが表示されていることを確認(草0)
  7. ログアウト
  8. komagataでログイン
  9. http://localhost:3000/users/459775584に遷移
  10. 画面中央付近にGithub Contributionのテーブルが用事されていることを確認(草あり)

Screenshot

変更前

image

変更後

  • hatsuno(一般ユーザー)
    image

  • komagata(管理者)
    image

補足

  • Todo Designとコメントが入っている箇所はレビュー後にmachidaさんへ依頼予定

@yuma-matsui yuma-matsui changed the title Hotfix/fix GitHub grass GithubのContributionが表示されないバグを修正した Jul 20, 2023
@yuma-matsui yuma-matsui marked this pull request as ready for review July 20, 2023 11:39
@yuma-matsui
Copy link
Contributor Author

@wata00913

お疲れ様です!
こちらのコードレビューをお願いします。

@wata00913
Copy link
Contributor

@yuma-matsui
お疲れ様です!
レビュー依頼ありがとうございます。
来週月曜までに返信出来そうですが、大丈夫でしょうか?

@yuma-matsui
Copy link
Contributor Author

ありがとうございます!
いつでも大丈夫です🙆‍♂️

Copy link
Contributor

@wata00913 wata00913 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GitHubの草とBootcampアプリで表示する草が、一致してない箇所がありました。

  • Bootcampのページ
    image
  • GitHubのページ
    image

@name = name
end

def generate_table
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

関数名がgenerate_tableだと、サーバー側でHTML形式のテーブルやReactコンポーネントを作成したと勘違いしそうです。
generate_table_data(一例です)など他の命名にした方が良いと思いました。

@wata00913
Copy link
Contributor

wata00913 commented Jul 24, 2023

GitHubのページと同様に以下の項目を追加した方が良さそうです。

  • 列ヘッダー(月)
    image
  • 行ヘッダー(曜日)
    image
  • 草の補足説明
    image

@wata00913
Copy link
Contributor

@yuma-matsui
お待たせしました〜!
動作確認はOKでした。

いくつかコメントしましたので、確認をよろしくお願いします🙇‍♂️

@yuma-matsui
Copy link
Contributor Author

ありがとうございます!
今週は対応が難しいので次週対応進めます!

@komagata
Copy link
Member

@yuma-matsui こちらいかがでしょうか?
緊急度が高いのでもし作業が難しければ私の方で別途進めたいと思います。

@yuma-matsui
Copy link
Contributor Author

@komagata
作業が滞ってしまい申し訳ありません。
対応が難しいのでご対応いただいてもよろしいでしょうか。

@komagata
Copy link
Member

@yuma-matsui かしこまりました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants