-
-
Notifications
You must be signed in to change notification settings - Fork 326
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: urls for js libs on print.html #2185
Conversation
Closes lnbits/withdraw#22 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #2185 +/- ##
=======================================
Coverage 59.28% 59.28%
=======================================
Files 59 59
Lines 8864 8864
=======================================
Hits 5255 5255
Misses 3609 3609 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we also need to fix the CSS:
diff --git a/lnbits/templates/print.html b/lnbits/templates/print.html
index ef6beb4a..4fde0878 100644
--- a/lnbits/templates/print.html
+++ b/lnbits/templates/print.html
@@ -3,7 +3,7 @@
<html lang="en">
<head>
{% for url in INCLUDED_CSS %}
- <link rel="stylesheet" type="text/css" href="{{ url }}" />
+ <link rel="stylesheet" type="text/css" href="{{ static_url_for('static', url) }}" />
{% endfor %}
<style>
@page {
fab648d
to
07c01ab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CSS fixed in 07c01ab
This fixes, for example, printing LNURLw vouchers.