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

メンターのカテゴリー画面のカテゴリー作成ボタンの移動 #6729

Merged
merged 2 commits into from
Aug 1, 2023

Conversation

rira100000000
Copy link
Contributor

@rira100000000 rira100000000 commented Jul 15, 2023

Issue

概要

カテゴリーページのカテゴリー作成ボタンの位置をタブの下に移動しました。

変更確認方法

  1. ブランチfeature/move-category-new-buttonをローカルに取り込む
  2. foreman start -f Procfile.devでローカル環境を立ち上げる
  3. ブラウザでhttp://localhost:3000を開く
  4. 管理者でログインする
  5. http://localhost:3000/mentor/categoriesにアクセスする
  6. カテゴリー作成ボタンの位置を確認する

Screenshot

変更前

image

変更後

image

@rira100000000
Copy link
Contributor Author

@machida
お疲れ様です。
該当箇所の変更が完了しましたので、画面のデザインをお願いします。

@machida
Copy link
Member

machida commented Jul 17, 2023

@rira100000000 了解ですー

@machida machida force-pushed the feature/move-category-new-button branch from 0b8933b to 19c9e29 Compare July 26, 2023 05:12
@machida
Copy link
Member

machida commented Jul 26, 2023

@rira100000000 お待たせしました🙏 対応しました。

最新の main を取り込んだので、手元の環境で

git pull --rebase origin feature/move-category-new-button

をお願いします。

@machida machida removed their assignment Jul 26, 2023
@machida
Copy link
Member

machida commented Jul 26, 2023

確認をしたら受講生レビューにすすめてくださいー

@rira100000000 rira100000000 marked this pull request as ready for review July 27, 2023 12:49
@rira100000000
Copy link
Contributor Author

@Kassy0220
お疲れ様です!
こちらレビューをお願いしたいのですが、ご都合いかがでしょうか?
(急ぎではありません)

Copy link
Contributor

@Kassy0220 Kassy0220 left a comment

Choose a reason for hiding this comment

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

@rira100000000
お疲れ様です🍵

メンターのカテゴリー一覧画面のカテゴリー作成ボタンの移動を確認しました。
コードの変更点は大丈夫だと思うので、私からはApproveさせていただきます〜🙆‍♂️

すみません、コードとは関係ないのですが一点だけいいでしょうか?
元のIssueの概要が管理者でログイン http://localhost:3000/admin/categories にアクセス。となっているので、URLをhttp://localhost:3000/mentor/categoriesと変更されると、IssueとPRの間の齟齬がなくなるのではないかと思いました。

@rira100000000
Copy link
Contributor Author

@Kassy0220
お疲れ様です!
Issueの修正いたしました!
迅速な対応ありがとうございます😄

@rira100000000
Copy link
Contributor Author

@komagata
チームメンバーからApproveを頂いたので、レビューをお願いいたします!

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 7d83e09 into main Aug 1, 2023
5 checks passed
@komagata komagata deleted the feature/move-category-new-button branch August 1, 2023 04:51
@github-actions github-actions bot mentioned this pull request Aug 1, 2023
16 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.

4 participants