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

つなぎこみをしまくる #120

Merged
merged 11 commits into from
Jan 25, 2025
Merged

つなぎこみをしまくる #120

merged 11 commits into from
Jan 25, 2025

Conversation

SSlime-s
Copy link
Contributor

このプルリクエストには、クライアントサイドAPI処理の大幅な更新、InputMessageコンポーネントのリファクタリング、およびサーバーサイドCORS設定が含まれています。最も重要な変更点は、転送ロジックの統合、APIフックのリファクタリング、InputMessageコンポーネントの機能強化です。

クライアントサイドAPI処理:

  • 各ファイルで新しい転送インスタンスを作成する代わりに、client/src/api/transport.tsからの共有TRANSPORTインスタンスを使用するようにAPIフックをリファクタリングしました。この変更により、コードが簡素化され、アプリケーション全体の転送設定の一貫性が確保されます。[1] [2] [3] [4] [5] [6]
  • ミューテーションにuseSWRMutationを使用するようにAPIフックを更新し、可読性と保守性を向上させるため、フェッチャー関数を改善しました。[1] [2] [3] [4] [5]
  • 共有TRANSPORTインスタンスをエクスポートする新しいファイルclient/src/api/transport.tsを追加しました。

コンポーネントのリファクタリング:

  • イベントハンドラーにuseCallbackを使用するようにInputMessageコンポーネントをリファクタリングし、ローディング状態管理によるメッセージ送信ロジックを追加しました。これにより、複数回送信を防ぎ、パフォーマンスとユーザーエクスペリエンスが向上します。
  • メッセージ送信中にsendMessageボタンを無効にするように更新し、role属性とaria-label属性を追加することでアクセシビリティを向上させました。

サーバーサイド設定:

  • 特定のオリジンからのリクエストを許可し、資格情報を処理するために、server/src/router.rsにCORS設定を追加しました。この変更により、フロントエンドがCORSの問題なくバックエンドと通信できるようになります。[1] [2]

その他の変更点:

  • アンダーバーで始まる未使用の変数を無視する新しいESLintルールを追加しました。
  • 動的なフィールドサイズを使用し、位置クランプロジックを改善するために、client/src/pixi/Canvas.tsxclient/src/pixi/World.tsxを更新しました。[1] [2] [3] [4] [5]
  • Jotaiを使用してユーザーの位置状態を管理するために、client/src/state/userPosition.tsに新しいuserPositionAtomを導入しました。

これらの変更は、コードベースの保守性、パフォーマンス、および機能を総合的に向上させます。

@SSlime-s SSlime-s requested a review from ikura-hamu January 25, 2025 22:41
@SSlime-s SSlime-s self-assigned this Jan 25, 2025
Copy link
Member

@ikura-hamu ikura-hamu left a comment

Choose a reason for hiding this comment

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

たぶんいいと思います

@SSlime-s SSlime-s merged commit 2193c4e into main Jan 25, 2025
7 checks passed
@SSlime-s SSlime-s deleted the SSlime/attatch-message branch January 25, 2025 22:49
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