devlinkcrew / next-netlify-starter

0 stars 1 forks source link

Feat/dev ueda #44

Closed towaueda closed 7 months ago

towaueda commented 9 months ago

コンタクト画面のエラーメッセージをアラートではなくバリデーションエラーに修正 全ページの空白の修正 プライバシーポリシーが外部遷移とわかりやすいように画像の差し込み

linkcrew commented 9 months ago

Deploy Preview for clever-croquembouche-cbb16c ready!

Name Link
Latest commit 09811527406ef5bb3335d0deff03ac3de2c65487
Latest deploy log https://app.netlify.com/sites/clever-croquembouche-cbb16c/deploys/65c0eab29ab7000008adfc71
Deploy Preview https://deploy-preview-44--clever-croquembouche-cbb16c.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] commented 9 months ago

Deploy Preview for gilded-salmiakki-3b0ce3 ready!

Name Link
Latest commit 09811527406ef5bb3335d0deff03ac3de2c65487
Latest deploy log https://app.netlify.com/sites/gilded-salmiakki-3b0ce3/deploys/65c0eab2fe56b4000866c51e
Deploy Preview https://deploy-preview-44--gilded-salmiakki-3b0ce3.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

HaraRyonosuke commented 9 months ago

コード上は問題なかったのですが、私からは1点だけ修正依頼をお願いしたいです。 image

開発だけでなく範囲外のキャリア支援についてもそうですが見出しが上の最初のハードルを越えられるようお手伝い致します。などの文言と近くなって目立ちにくくなっている気がするのでもう少し間隔を開けて見出し感を強めていただけるとありがたいです。

yyysakai commented 9 months ago

@towaueda すみません、レビューコメント残したつもりが更新されていませんでした...

対応いただきありがとうございます! 修正点は原さんの指摘を参考にしていただき、私からは追加でissueの対応お願いできますでしょうか?

towaueda commented 9 months ago

@HaraRyonosuke san @yyysakai san 修正しましたのでご確認よろしくお願いします。

yyysakai commented 8 months ago

daisyUIで作成されたコンポーネントがheaderの上を通っていました。こちら永遠さんの方では起きてない現象ですか??

スクリーンショット 2024-03-17 21 16 33 スクリーンショット 2024-03-17 21 16 42
yyysakai commented 8 months ago

@towaueda レビュー担当者が動けなくなった影響でレビュー遅れてしまい申し訳ありませんでした...

指摘入れましたが、ほとんどLGTMです!コメント部分が直ればマージしてしまおうと思います!

daisyUIで作成されたコンポーネントがheaderの上を通っていました。こちら永遠さんの方では起きてない現象ですか??

お時間経ってしまい申し訳ないのと、微小な修正なので、上記の回答だけいただければ私の方で修正対応致します🙇

towaueda commented 8 months ago

@yyysakai san こちらすべて修正しましたのでご確認よろしくお願いいたします。

daisyUIで作成されたコンポーネントがheaderの上を通っていました。こちら永遠さんの方では起きてない現象ですか??

こちらも起きていたので修正しました。