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

memo.vue関連のファイルを削除する #7459

Merged
merged 1 commit into from
Mar 9, 2024
Merged

Conversation

a-kuroki-gs
Copy link
Contributor

@a-kuroki-gs a-kuroki-gs commented Feb 29, 2024

Issue

概要

Issueの内容はmemo.vueの非Vue化であったが、
memo.vueは座席予約システムの廃止に伴い、現在使われていないことがわかったため削除する。

ref: #1230 #3203

変更確認方法

  1. feature/delete-memo-vueをローカルに取り込む。
  2. memo.vue関連のファイルがきちんと削除・変更されていることを確認する(上記記載のPRを参考)。
  3. アプリの動きに影響がないことも念のため確認いただきたいです。

Screenshot

画面の変更はありません。

@a-kuroki-gs a-kuroki-gs self-assigned this Feb 29, 2024
@a-kuroki-gs a-kuroki-gs marked this pull request as ready for review February 29, 2024 01:10
@a-kuroki-gs
Copy link
Contributor Author

@goruchanchan
お疲れ様です!
こちらのPRのレビューをお願いしてもよろしいでしょうか?
お時間厳しければ遠慮なくおっしゃってください🙌

ご確認よろしくお願いいたします🙏

@goruchanchan
Copy link
Contributor

@a-kuroki-gs お疲れ様です!レビュー依頼ありがとうございます!一週間程度をめどに確認いたしますので、お待ちください🙇‍♂️

@goruchanchan
Copy link
Contributor

goruchanchan commented Mar 3, 2024

@a-kuroki-gs お疲れ様です!確認実施しました!

アプリの動きに影響がないことも念のため確認いただきたいです。

上記の保証がなかなか難しいと思いましたが、今回の memo 関連で検索かけて関連しそうなコートが残ってないように思いましたので問題なしと判断します🙇‍♂️

コンフリクトが発生しているようなので、そちらの対応だけお願いします🙇‍♂️

座席予約システムの導入で作成されたが、廃止に伴い現在使われていないため
@a-kuroki-gs a-kuroki-gs force-pushed the feature/delete-memo-vue branch from ec336a4 to d7c553f Compare March 6, 2024 02:56
@a-kuroki-gs
Copy link
Contributor Author

@goruchanchan

アプリの動きに影響がないことも念のため確認いただきたいです。

上記の保証がなかなか難しいと思いましたが、今回の memo 関連で検索かけて関連しそうなコートが残ってないように思いましたので問題なしと判断します🙇‍♂️

難しい依頼でしたが、ご確認いただきありがとうございます🙌
コンフリクトについても解消しております。

@komagata
お疲れ様です!
メンバーにApproveをいただいたので、レビューをお願いいたします🙏

Copy link
Contributor

@goruchanchan goruchanchan left a comment

Choose a reason for hiding this comment

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

コンフリクト対応確認できましたので承認いたします🙇‍♂️

@a-kuroki-gs
Copy link
Contributor Author

@goruchanchan
ご確認いただきありがとうございます🙌

@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 70ee9a2 into main Mar 9, 2024
3 checks passed
@komagata komagata deleted the feature/delete-memo-vue branch March 9, 2024 17:09
@github-actions github-actions bot mentioned this pull request Mar 9, 2024
29 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