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

1678: Change filename of single card pdf #1778

Merged
merged 5 commits into from
Nov 23, 2024

Conversation

ztefanie
Copy link
Member

@ztefanie ztefanie commented Nov 15, 2024

Short description

Adjusted pdf filenames for single cards to owner name for nuernberg and bayern

Side effects

none

Testing

Create a single card in nuernberg or bayern -> filename should be firstname lastname.pdf
create multiple cards in nuernberg or bayern -> filename should stay as it is berechtigungskarten.pdf
koblenz card creation should stay as it is

Resolved issues

Fixes: #1678 #1756

Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

Change in average Code Health of affected files: -0.01 (9.19 -> 9.18)

  • Declining Code Health: 1 findings(s) 🚩

  • Affected Hotspots: 1 files(s) 🔥

View detailed results in CodeScene

Copy link
Contributor

@f1sh1918 f1sh1918 left a comment

Choose a reason for hiding this comment

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

LGMT 👍

@ztefanie ztefanie merged commit 8c73d29 into main Nov 23, 2024
2 checks passed
@ztefanie ztefanie deleted the 1678-change-filename-of-single-card-pdf branch November 23, 2024 08:24
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.

Change filename of the activation-pdf
4 participants