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

[FE] シュミレーターのRideStatus表示のうち"完了"を"空車"として表示する / 他 #612

Merged
merged 7 commits into from
Dec 5, 2024

Conversation

popunbom
Copy link
Contributor

@popunbom popunbom commented Dec 4, 2024

relates: #241

下記に対応しました

シュミレーターのRideStatus表示のうち"完了"を"空車"として表示する

評価前 → 評価後
arrived completed

  • 「完了」ではなく「空車」と表示されるようになった

@popunbom
Copy link
Contributor Author

popunbom commented Dec 4, 2024

@narirou @imamiya-masaki レビューお願いいたします。

@popunbom popunbom mentioned this pull request Dec 4, 2024
23 tasks
Copy link
Contributor

@imamiya-masaki imamiya-masaki left a comment

Choose a reason for hiding this comment

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

rideStatus === "COMPLETED"を、FE上ではundefinedで扱うようにしていただきたいです 🙏

@popunbom popunbom force-pushed the frontend/refactor/completed-as-matching branch from 591a255 to 068650d Compare December 4, 2024 13:01
@popunbom popunbom force-pushed the frontend/refactor/completed-as-matching branch from 068650d to 7c53ea8 Compare December 4, 2024 13:04
@popunbom popunbom changed the title [FE] シュミレーターのRideStatus表示のうち"完了"を"空車"として表示する [FE] シュミレーターのRideStatus表示のうち"完了"を"空車"として表示する / 他 Dec 4, 2024
@popunbom
Copy link
Contributor Author

popunbom commented Dec 4, 2024

@narirou @imamiya-masaki

ペアプロありがとうございました。
本日の作業内容を整理しました。

  • da0031d: COMPLETED のときのステータス表示を「空車」にする
  • 8f7c378: COMPLETED のときのシミュレーターの椅子位置をリセットする
  • e66b21f: 評価完了後に推定運賃をリセットする
  • 17e20e4: 賃走時のシミュレーターの進行方向を修正
  • 7c53ea8: リファクタリング

@narirou narirou merged commit bc7b293 into main Dec 5, 2024
1 check passed
@narirou narirou deleted the frontend/refactor/completed-as-matching branch December 5, 2024 00:06
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