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

Feat manual payments controller #3045

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

ivannovosad
Copy link
Contributor

Roadmap Task

👉 https://getlago.canny.io/feature-requests/p/log-partial-payments

Context

This PR introduces a new endpoint to list all payments. Additionally, it adds the ability to filter payments by a specific invoice_id, allowing users to narrow down the results to payments associated with a particular invoice. This enhancement improves the flexibility and usability of the payments listing functionality.
It also adds create and show actions in the PaymentsController.

Description

Added Endpoint:

  • Created an endpoint to list, show and create payments.
  • Filter by invoice_id:
  • Added functionality to filter payments based on the provided invoice_id.
  • Included a validation step to ensure the invoice_id is in the correct UUID format before executing the query.

Query Updates:

  • Refactored PaymentsQuery to handle dynamic filtering by invoice_id.
  • Ensured that the query properly joins related tables, such as invoices and payment_requests, to support filtering.

ivannovosad and others added 2 commits January 10, 2025 08:38
## Context

This PR introduces a new endpoint to list all payments. Additionally, it
adds the ability to filter payments by a specific invoice_id, allowing
users to narrow down the results to payments associated with a
particular invoice. This enhancement improves the flexibility and
usability of the payments listing functionality.

## Description

**Added Endpoint:**

- Created an endpoint to list all payments.
- Filter by invoice_id:
- Added functionality to filter payments based on the provided
invoice_id.
- Included a validation step to ensure the invoice_id is in the correct
UUID format before executing the query.

**Query Updates:**

- Refactored PaymentsQuery to handle dynamic filtering by invoice_id.
- Ensured that the query properly joins related tables, such as invoices
and payment_requests, to support filtering.
@ivannovosad ivannovosad force-pushed the feat-manual-payments-controller branch from 333db63 to e4efd7b Compare January 10, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants