Skip to content

Dev#28

Merged
Ogunaru merged 12 commits intomainfrom
dev
Mar 4, 2025
Merged

Dev#28
Ogunaru merged 12 commits intomainfrom
dev

Conversation

@Ogunaru
Copy link
Member

@Ogunaru Ogunaru commented Feb 10, 2025

This pull request introduces several significant changes to the booth_checker project, including the addition of OpenAI integration, refactoring of existing functions, and enhancements to the logging and error handling mechanisms. The most important changes are grouped by their themes below.

OpenAI Integration:

  • Added chatgpt import and integrated OpenAI API for generating summaries of changes in booth_checker/__main__.py. [1] [2] [3]
  • Created a new file booth_checker/chatgpt.py to handle interactions with the OpenAI API.

Function Refactoring:

  • Modified the init_update_check function to reorder parameters and add a new summary_this parameter.
  • Removed the build_tree and tree_to_html functions, and replaced them with a more streamlined approach. [1] [2]

Logging and Error Handling:

  • Enhanced logging to include order_num in various log messages for better traceability. [1] [2] [3]
  • Updated the send_error_message function to include order_num in the log messages.

Data Handling:

  • Added new fields name-list and summary to the JSON structures and data handling functions. [1] [2] [3]
  • Updated the booth_checker/booth_sqlite.py to include the new summary_this field in the database queries.

Discord Bot Enhancements:

  • Modified the booth_discord/booth_discord.py to handle the new summary_this parameter and include summaries in the messages sent to Discord. [1] [2] [3] [4]

@Ogunaru Ogunaru requested a review from Nopied February 10, 2025 00:29
@Ogunaru Ogunaru self-assigned this Feb 10, 2025
@Nopied
Copy link
Collaborator

Nopied commented Mar 4, 2025

Lookin good (not sure actually)

@Ogunaru Ogunaru merged commit 082ca5a into main Mar 4, 2025
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