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

ナビゲーションデバッグ画面にTextFieldを配置 #293

Merged
merged 3 commits into from
Jul 31, 2024

Conversation

mj-hd
Copy link
Contributor

@mj-hd mj-hd commented Jul 26, 2024

概要

(元々はmacrosなどでうまいことGoRouteDataからサンプルパラメータを持って来れないかと試してたのですが、挫折してしまいました...手動入力の形で対応です)

レビュー観点

  • UIや体験が微妙かもしれません。また、他の妙案もあるかもしれません。ぜひご指摘いただけると幸いです

レビューレベル

  • Lv1: ぱっとみて違和感がないかチェックして Approve する
  • Lv2: 仕様レベルまで理解して、仕様通りに動くかある程度検証して Approve する
  • Lv3: 実際に環境で動作確認したうえで Approve する

レビュー優先度

  • すぐに見てもらいたい ( hotfix など ) 🚀
  • 今日中に見てもらいたい 🚗
  • 今日〜明日中で見てもらいたい 🚶
  • 数日以内で見てもらいたい 🐢

画像 / 動画

Screenshot_1721993176

確認したこと

  • 適当なQuery/Path Parameterが必要なルートに対して、TextFieldで書き換えた上で画面遷移できること

動作確認手順

  • Debug Mode画面へ遷移し、Go Navigation Debug Pageを選択
  • 適当な選択肢を選び、TextFieldを書き換えたのちにGoボタンをおす

備考

@github-actions github-actions bot added the @packages/features/debug_mode packages features debug_mode package label Jul 26, 2024
@mj-hd mj-hd marked this pull request as ready for review July 26, 2024 11:30
@yumemi-team-review-requester yumemi-team-review-requester bot requested review from a team, rizumita and K9i-0 and removed request for a team July 26, 2024 11:30
Copy link

Ready for review 🚀

@mj-hd mj-hd requested a review from blendthink July 26, 2024 11:30
Copy link

github-actions bot commented Jul 26, 2024

Visit the preview URL for this PR (updated for commit 40ee76f):

https://flutter-mobile-project-template-catalog--pr293-debug-p-efyjb4s7.web.app

(expires Wed, 07 Aug 2024 05:38:09 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 9ea56735a63d07a7cfe62eb204b0528284c37c23

@blendthink
Copy link
Member

明日レビュー予定です 🙏

Copy link
Member

@blendthink blendthink left a comment

Choose a reason for hiding this comment

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

@mj-hd
いくつかコメントさせていただきましたのでご確認お願いします 🙏

Comment on lines 103 to 106
var routePath = routeBase.path;
if (parentRoute != null) {
routePath = p.join(parentRoute.path, routePath);
}
Copy link
Member

Choose a reason for hiding this comment

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

good-badge
処理がシンプルになっていいですね!
ご対応ありがとうございます!

@blendthink blendthink enabled auto-merge July 31, 2024 05:37
Copy link
Member

@blendthink blendthink left a comment

Choose a reason for hiding this comment

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

LGTM です!
ご対応ありがとうございました!

@blendthink blendthink merged commit 6546756 into main Jul 31, 2024
13 checks passed
@blendthink blendthink deleted the debug-params branch July 31, 2024 05:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@packages/features/debug_mode packages features debug_mode package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants