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

fix(pdf): Remove extra blank link in custom footer #1854

Merged
merged 1 commit into from
Apr 9, 2024

Conversation

julienbourdeau
Copy link
Contributor

Context

An organisation can choose to display a custom text block on invoices.

Description

With this change, we'll ensure the all the blank line before and after the last lines are removed. We'll also remove extra blank lines.

Example

CONFIG

Screenshot 2024-04-08 at 14 05 30

BEFORE

Screenshot 2024-04-08 at 13 58 00

AFTER

Screenshot 2024-04-08 at 13 58 57

@julienbourdeau julienbourdeau self-assigned this Apr 8, 2024
@julienbourdeau julienbourdeau merged commit c7e7cd8 into main Apr 9, 2024
6 checks passed
@julienbourdeau julienbourdeau deleted the fix/blank-line-footer branch April 9, 2024 09:22
drejc pushed a commit to fliqa-io/lago-api that referenced this pull request May 15, 2024
## Context

An organisation can choose to display a custom text block on invoices.

## Description

With this change, we'll ensure the all the blank line before and after
the last lines are removed. We'll also remove extra blank lines.

### Example

#### CONFIG

<img width="1021" alt="Screenshot 2024-04-08 at 14 05 30"
src="https://github.com/getlago/lago-api/assets/1525636/e44acb32-fede-404b-9f69-5fe7e6704a2f">

#### BEFORE

<img width="851" alt="Screenshot 2024-04-08 at 13 58 00"
src="https://github.com/getlago/lago-api/assets/1525636/890b58b4-735b-442b-b843-7efee1a6dff2">

#### AFTER

<img width="855" alt="Screenshot 2024-04-08 at 13 58 57"
src="https://github.com/getlago/lago-api/assets/1525636/a19971ee-1013-42fc-9419-e1f1971bf188">
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.

None yet

2 participants