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

Eventの空白をzodでバリデーション #116

Merged
merged 2 commits into from
Dec 11, 2024
Merged

Eventの空白をzodでバリデーション #116

merged 2 commits into from
Dec 11, 2024

Conversation

Inlet-back
Copy link
Contributor

@Inlet-back Inlet-back commented Dec 6, 2024

close: #101

概要

カレンダーからイベント等をいれる際に空白のまま入れられる問題を解消

変更内容

event-creator.tsx内にzodスキーマの定義と適応

テスト方法

フロント側での目視での確認。

確認項目

  • コードが正しく動作していること
  • ドキュメントやコメントが適切であること

その他

バックエンドで使う流れでzodでバリデーションしたけど、他に使わずおもそうなら別のやり方でやります
vercelのトークンが古いみたいで新しいのでvercel pull --yes --environment=preview --token=*とvercel loginしたけどうまくいかなかったので後でやります

@Inlet-back Inlet-back requested a review from yuto-trd December 6, 2024 10:19
@Inlet-back Inlet-back changed the title Eventの空白をzodでバリデーション Closes #101 Eventの空白をzodでバリデーション Dec 6, 2024
@yuto-trd yuto-trd merged commit 541f5c3 into main Dec 11, 2024
2 of 3 checks passed
@yuto-trd yuto-trd deleted the feature/#101 branch December 11, 2024 12:50
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.

タイトルを入力していないタスクが追加されてしまう
2 participants