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: allow to add fee on draft invoices #2004

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

Conversation

ansmonjol
Copy link
Collaborator

@ansmonjol ansmonjol commented Jan 29, 2025

Context

We are currently able to edit a fee on a draft invoice.
It's convenient as all fees (including ones with 0 unit and amount cents) are all created at billing time.

However we recently removed the creation of "0-fees" mostly for performance gain (they were representing a huge amount of the created fees every billing cycle)

Description

This PR does

  • Refactor the edit fee drawer to follow new design guidelines
  • Add the possibility to create a fee within the same drawer, in case none is passed in props.

Better reviewing the 2 commits separately.

Fixes LAGO-694

@ansmonjol ansmonjol added the 🛠 feature New feature or request label Jan 29, 2025
@ansmonjol ansmonjol self-assigned this Jan 29, 2025
@ansmonjol ansmonjol force-pushed the feat-adjust-zero-fees-qa branch 2 times, most recently from e433c0c to d1739e5 Compare January 29, 2025 16:17
@ansmonjol ansmonjol force-pushed the feat-adjust-zero-fees-qa branch 2 times, most recently from ea0d6f2 to c86c9e9 Compare January 29, 2025 16:54
@ansmonjol ansmonjol force-pushed the feat-adjust-zero-fees-qa branch from c86c9e9 to a4b5f96 Compare January 29, 2025 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants