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

ポートフォリオページの見出し文言変更 #7929

Merged
merged 2 commits into from
Jul 19, 2024

Conversation

Judeeeee
Copy link
Contributor

@Judeeeee Judeeeee commented Jul 4, 2024

Issue

概要

ポートフォリオ一覧ページの見出し文言を変更しました。

  • 修正前
    • ポートフォリオ
  • 期待する表示
    • みんなのポートフォリオ

変更確認方法

  1. bug/update-portfolio-headings-textをローカルに取り込む
  2. foreman start -f Procfile.devでローカルサーバーを立ち上げる
  3. メンター or 管理者 or 受講生アカウントでログインする
  4. ポートフォリオ一覧ページにアクセスする

確認事項

ページの見出しがみんなのポートフォリオになっている

Screenshot

変更前

スクリーンショット 2024-07-05 22 31 36

変更後

スクリーンショット 2024-07-05 22 33 27

@Judeeeee Judeeeee self-assigned this Jul 4, 2024
@Judeeeee Judeeeee marked this pull request as draft July 4, 2024 12:51
@Judeeeee Judeeeee marked this pull request as ready for review July 5, 2024 13:06
@Judeeeee
Copy link
Contributor Author

Judeeeee commented Jul 5, 2024

@motohiro-mm

お疲れ様です!
お忙しいところ恐縮ですが、こちらのPRのレビューをお願いできますでしょうか👀
ご都合がつかなければ遠慮なく仰っていただけると幸いですー🙏

@motohiro-mm
Copy link
Contributor

@Judeeeee

承知いたしました!
できるだけ早めにレビューさせていただきますが、他のレビューもあるので、1週間をめどにレビューさせてください🙏
もしお急ぎの場合は、申し訳ありませんが他の方にお願いしていただければと思いますので、その際はご連絡ください🙇‍♀️

あと、このPRのdescription内のScreenshot画像が現在見れなかったので確認をお願いいたします🙏
よろしくお願いいたします!

@Judeeeee
Copy link
Contributor Author

Judeeeee commented Jul 5, 2024

@motohiro-mm
早速の回答助かります!ありがとうございます🙏
特に急ぎではないので、motohiroさんのご都合の良いタイミングで進めていただけると幸いです〜!

あと、このPRのdescription内のScreenshot画像が現在見れなかったので確認をお願いいたします🙏

ご指摘ありがとうございます!
シークレットモードで確認したところ、確かに画像が確認できませんでした😓
すぐに差し替えさせていただきますね!✨

@motohiro-mm
Copy link
Contributor

@Judeeeee

Screenshot見れました!ありがとうございます👍
お手隙の際にReviewersに私を追加しておいてください!
レビューしましたら再度ご連絡します!

@Judeeeee Judeeeee requested a review from motohiro-mm July 6, 2024 02:49
Copy link
Contributor

@motohiro-mm motohiro-mm left a comment

Choose a reason for hiding this comment

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

確認させていただきました!
問題ないと思いましたので、私からはapproveさせていただきます!

@Judeeeee
Copy link
Contributor Author

Judeeeee commented Jul 8, 2024

@motohiro-mm
承知しました〜!
確認ありがとうございました🙏

@Judeeeee
Copy link
Contributor Author

Judeeeee commented Jul 8, 2024

@komagata

お疲れ様ですー!
チームメンバーからapproveいただいたので、お手隙の際にレビューしていただけると幸いです🙏

@Judeeeee Judeeeee requested a review from komagata July 8, 2024 12:41
@@ -6,7 +6,7 @@ header.page-header
.page-header__inner
.page-header__start
h2.page-header__title
= title
|みんなのポートフォリオ
Copy link
Member

Choose a reason for hiding this comment

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

全体で下記のようにスペースを開けているので同じようにしていただければありがたいです〜

Suggested change
|みんなのポートフォリオ
| みんなのポートフォリオ

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@komagata

e214316で修正いたしましたのでご確認ください🙏

@Judeeeee Judeeeee force-pushed the bug/update-portfolio-headings-text branch from eabae4a to e214316 Compare July 16, 2024 13:41
@Judeeeee Judeeeee requested a review from komagata July 16, 2024 14:03
Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

確認させて頂きました。OKです〜🙆‍♂️

@komagata komagata merged commit 8d708ca into main Jul 19, 2024
4 checks passed
@komagata komagata deleted the bug/update-portfolio-headings-text branch July 19, 2024 07:29
@github-actions github-actions bot mentioned this pull request Jul 19, 2024
13 tasks
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.

3 participants