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

added support for prices including taxes for #1

Merged
merged 1 commit into from Sep 29, 2015
Merged

Conversation

thilo
Copy link
Member

@thilo thilo commented Sep 29, 2015

invoice reminders. Also refactored the code to
have a more homogeneous interface for plan and extas

invoice reminders. Also refactored the code to
have a more homogeneous	interface for plan and extas
@@ -10,12 +10,12 @@ def default_body
Extras: {% for extra in plan.extras %}
{{extra.name}}: {{extra.price | money }} {{plan.currency}}
{% endfor %}{% endif %}
Price: {{plan.price_per_cycle | money}} {{plan.currency}}.
Price: {{plan.price | money}} {{plan.currency}}.
Copy link
Member

Choose a reason for hiding this comment

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

wenn du den namen des attributes änderst, gehen alle bestehenden reminder kaputt. gibt's da noch nen alias drauf?

langalex added a commit that referenced this pull request Sep 29, 2015
added support for prices including taxes for
@langalex langalex merged commit 6cc3f2b into master Sep 29, 2015
@langalex langalex deleted the tax_support branch September 29, 2015 14:26
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