Skip to content

Change BVR to QR bills#41

Open
TeoGoddet wants to merge 6 commits intoArcaniteSolutions:masterfrom
TeoGoddet:feat/qr-bill
Open

Change BVR to QR bills#41
TeoGoddet wants to merge 6 commits intoArcaniteSolutions:masterfrom
TeoGoddet:feat/qr-bill

Conversation

@TeoGoddet
Copy link
Copy Markdown
Contributor

Do not review before #32
Depends on #32

@TeoGoddet TeoGoddet changed the title Change BVR to QR bills WAIT : Change BVR to QR bills Aug 24, 2020
@the-glu
Copy link
Copy Markdown
Member

the-glu commented Aug 28, 2020

@TeoGoddet J'ai l'impression que la MR est toujours mixée avec l'autre (ou y'a plus de change que pour juste le QRCode), t'arrives à regarder ou confirmer que c'est nromal ?

@TeoGoddet TeoGoddet changed the title WAIT : Change BVR to QR bills Change BVR to QR bills Aug 30, 2020
@TeoGoddet
Copy link
Copy Markdown
Contributor Author

@the-glu Il devait y avoir les commit de #32, Maintenant c'est sur que c'est correct

{% if invoice.display_qr %}
<pdf:nexttemplate name="qr" />
<pdf:nextpage>
<div id=qr_comments>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

guimmets

}

.pay_title
{
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

c'est pas très consistant avec le reste (aka met le { sur la ligne d'en dessus)


<small>{% trans "Attention: Ce BVR ne peut pas être utilisé pour un paiement aux guichets de la poste ou d'une banque" %}</small>
</center>
<p style="color: white">This is a fake placeholder</p>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

?

Fix qr invoices

Add migration for qr invoice

Correction

Address Help Text

Fix things

try to display qr on pdf

Switch to QR

Various improvements

Nicer invoices

Final style for invoices
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.

2 participants