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

⚙️ refactor: Enhance Logging, Navigation And Error Handling #5910

Merged
merged 7 commits into from
Feb 16, 2025

Conversation

danny-avila
Copy link
Owner

Summary

  • Replaced console logs with logger.warn and logger.error in the useNavigateToConvo hook
  • Ensured missing or removed endpoints are properly handled in the useNavigateToConvo hook
  • Updated the buildDefaultConvo utility to accept an optional endpoint parameter and reorganized parameter usage for clearer default conversation initialization.
  • Enhanced the logger utility by introducing a type parameter to standardize log formatting across different log levels.
  • Fixed the HeaderOptions component to prevent ModelSelect from rendering when using agent endpoints
  • Improved the logAxiosError function in the API utils by streamlining error handling with try-catch logic and reducing verbosity when no response is received.

Checklist:

  • My code adheres to this project's style guidelines
  • I have performed a self-review of my code
  • I have commented in complex areas for clarity
  • I have made pertinent documentation changes
  • My changes do not introduce new warnings
  • I have written tests demonstrating that my changes are effective
  • Local unit tests pass with my changes
  • Any changes dependent on mine have been merged and published in downstream modules

@danny-avila danny-avila changed the title ⚙️ refactor: Enhance Logging, Navigation And Error Handling Mechanisms ⚙️ refactor: Enhance Logging, Navigation And Error Handling Feb 16, 2025
@danny-avila danny-avila merged commit a65647a into main Feb 16, 2025
7 checks passed
@danny-avila danny-avila deleted the refactor/improvements branch February 16, 2025 16:47
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.

[Bug]: Unable to use endpoint switching when original endpoint is deleted
1 participant