Skip to content

Fixes and enhancements #53

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Fixes and enhancements #53

wants to merge 12 commits into from

Conversation

@sabelosimelane
Copy link

@sabelosimelane sabelosimelane commented Jan 23, 2025

Multiple Improvements and Bug Fixes

Key Changes

  1. Database Logging & Transaction Display

    • Added database logging for all incoming and outgoing requests
    • Implemented transaction display functionality in both DB and UI
    • Fixed handling of null messages in transaction display

  2. Model & Pricing Updates

    • Updated OpenAI model pricing
    • Fixed issues with unpopulated model data

  3. Technical Improvements

    • Enhanced proxy response handling and logging
    • Updated dependencies
    • Added package inclusion in pyproject.toml

Details

  • Implemented comprehensive request logging system
  • Fixed several data handling issues, particularly around null messages and model data
  • Improved system stability through better proxy response handling
  • Updated project dependencies and configuration

Testing

The changes can be tested by:

  1. Verifying transaction logging in the database
  2. Checking transaction display in the UI
  3. Confirming proper handling of requests with unpopulated model data
  4. Validating proxy responses and logging functionality

@sabelosimelane
updates
7346fd9
@sabelosimelane
udpates
35d8df7
@sabelosimelane
chore: Update dependencies and add package inclusion in pyproject.toml
98c33ff
@sabelosimelane
fix: improve proxy response handling and logging
83c4452 
- Fix chunked response handling to return proper JSON
- Add proper Content-Type and Content-Length headers
- Remove problematic transfer-encoding headers
- Enhance request/response logging for better debugging
- Fix double body read in request handling
@sabelosimelane
updates
6b2ca74
@sabelosimelane
Added logging to the database for all incoming and outgoing requests.
f16a9f6
@sabelosimelane
Now we can see transactions in the DB and th eUI
f7b5264
@sabelosimelane
fix: handle null messages in transaction display and add missing import
7b41379 
- Add null check in DisplayMessage component to prevent frontend errors
- Import deserialize_data in transaction repository to fix project view
@sabelosimelane
updated OpenAI model pricing
f3e126e
@sabelosimelane
fixed error with unpopulated model data
d8d45fe
@sabelosimelane
fixed error with unpopulated model data
4856e6c
@sabelosimelane
production ready fixes
3557151
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@sabelosimelane