Skip to content

Feat/bulk updates#10

Merged
RadCod3 merged 4 commits into
mainfrom
feat/bulk-updates
Sep 5, 2025
Merged

Feat/bulk updates#10
RadCod3 merged 4 commits into
mainfrom
feat/bulk-updates

Conversation

@RadCod3
Copy link
Copy Markdown
Owner

@RadCod3 RadCod3 commented Sep 5, 2025

Closes #6

Copilot AI review requested due to automatic review settings September 5, 2025 18:37
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements bulk transaction operations by adding functionality to create and update multiple transactions in a single operation. It replaces the specific budget-focused transaction update functionality with more general transaction update capabilities.

  • Adds bulk create and update operations for transactions
  • Replaces budget-specific transaction updates with general transaction updates
  • Updates test coverage for new bulk operations

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
tests/unit/test_server.py Adds tests for bulk create/update operations and replaces budget-specific update tests
tests/unit/test_models.py Removes tests for the deprecated UpdateTransactionBudgetRequest model
tests/unit/test_firefly_client.py Adds comprehensive tests for new bulk operations in the client layer
src/lampyrid/server.py Implements new MCP tools for bulk operations and general transaction updates
src/lampyrid/models/lampyrid_models.py Adds new request models for bulk operations and general transaction updates
src/lampyrid/clients/firefly.py Implements client methods for bulk create/update using individual API calls

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread src/lampyrid/clients/firefly.py Outdated
Comment thread src/lampyrid/clients/firefly.py
@RadCod3 RadCod3 merged commit e0d6952 into main Sep 5, 2025
@RadCod3 RadCod3 deleted the feat/bulk-updates branch September 5, 2025 18:41
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.

Add bulk transaction creation and update tools

2 participants