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

493/feature/select channel #526

Merged
merged 3 commits into from
Dec 29, 2024
Merged

Conversation

tktcorporation
Copy link
Owner

@tktcorporation tktcorporation commented Dec 29, 2024

This pull request includes several changes to improve error handling, command registration, and the main function configuration in the project. The most important changes include improvements to the SelectChannel command, enhancements to error reporting, and adjustments to the Sentry configuration.

Improvements to SelectChannel command:

Enhancements to error reporting:

  • src/error.rs: Added a new function report_error_with_cause to capture errors with their causes using Sentry, improving error diagnostics.

Adjustments to Sentry configuration:

  • src/main.rs: Enabled auto_session_tracking and debug options in the Sentry initialization for better monitoring and debugging.

Additional command registration:

  • src/handler/mod.rs: Added a comment to indicate the registration of new commands in the event handler.

- エラー報告時にエラーオブジェクトを直接送信するように変更

- report_error_with_cause関数を追加してエラーの詳細な情報を送信

- Sentryの初期化設定を改善してデバッグ情報を有効化
- チャンネル指定を任意に変更

- チャンネル未指定時は現在のチャンネルを使用

- エラーメッセージの改善
Copy link

@tktcorporation tktcorporation force-pushed the 493/feature/select-channel branch from fe97778 to 6af4292 Compare December 29, 2024 11:37
@tktcorporation tktcorporation marked this pull request as ready for review December 29, 2024 11:40
@tktcorporation tktcorporation merged commit 3e19e73 into master Dec 29, 2024
8 of 9 checks passed
@tktcorporation tktcorporation deleted the 493/feature/select-channel branch December 29, 2024 11:43
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.

1 participant