Skip to content

Commit

Permalink
Fix receipt check (#198)
Browse files Browse the repository at this point in the history
* Fix bekräftat since a new law

* Small fixes

* Update overview.html

* Fix translation in admin views and templates

* fipl

* good
  • Loading branch information
Adasjo authored Aug 14, 2024
1 parent fb9a7e3 commit e222dbc
Show file tree
Hide file tree
Showing 7 changed files with 22 additions and 28 deletions.
8 changes: 4 additions & 4 deletions admin/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,7 @@ def confirm_expense(request, pk):
expense = Expense.objects.get(pk=pk)

if not dauth.has_permission('confirm', request):
return HttpResponseForbidden("Du har inte rättigheterna för att bekräfta kvitton")
return HttpResponseForbidden("Du har inte rättigheterna för att bekräfta kvittons giltighet")

expense.confirmed_by = request.user
expense.confirmed_at = date.today()
Expand All @@ -218,7 +218,7 @@ def confirm_expense(request, pk):
comment = Comment(
expense=expense,
author=request.user.profile,
content='Jag bekräftar att kvittot finns i pärmen.'
content='Jag har bekräftat kvittots giltighet.'
)
comment.save()

Expand All @@ -233,7 +233,7 @@ def unconfirm_expense(request, pk):
expense = Expense.objects.get(pk=pk)

if not dauth.has_permission('unconfirm', request):
return HttpResponseForbidden("Du har inte rättigheterna för att avbekräfta kvitton")
return HttpResponseForbidden("Du har inte rättigheterna för att ta bort bekräftelse av kvittons giltighet")

expense.confirmed_by = None
expense.confirmed_at = None
Expand All @@ -242,7 +242,7 @@ def unconfirm_expense(request, pk):
comment = Comment(
expense=expense,
author=request.user.profile,
content='Jag tar bort bekräftelse i pärmen.'
content='Jag tar bort bekräftelsen av kvittots giltighet.'
)
comment.save()

Expand Down
4 changes: 2 additions & 2 deletions expenses/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -295,9 +295,9 @@ def status(self):
if self.is_attested() and self.confirmed_by:
return "Inväntar utbetalning"
if self.is_attested() and not self.confirmed_by:
return "Attesterad men inte i pärmen"
return "Attesterad men inte bekräftad av kassör"
if not self.is_attested() and self.confirmed_by:
return "Inte attesterad men i pärmen"
return "Inte attesterad men bekräftad av kassör"
return "Inte attesterad"

# Return the total amount of the expense parts
Expand Down
8 changes: 4 additions & 4 deletions templates/admin/confirm/overview.html
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
{% extends "../main.html" %}

{% block title %}Bekräfta kvitton i pärmen{% endblock %}
{% block title %}Bekräfta kvitton{% endblock %}

{% block content %}
<script src="https://cdnjs.cloudflare.com/ajax/libs/fetch/2.0.3/fetch.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/vue/2.5.2/vue.min.js"></script>
<div id="data">
<p>Här kan du hitta alla kvitton som inte bekräftats finns i pärmen.</p>
<p>Här kan du hitta alla kvitton som inte bekräftats.</p>
<label for="digital" style="padding-right: 0.5em;">Visa endast digitala</label><input v-model="digital" type="checkbox" id="digital" /><br />
<label for="analog" style="padding-right: 0.5em;">Visa endast analoga</label><input v-model="analog" type="checkbox" id="analog" />
<table v-if="expenses.length > 0">
Expand All @@ -24,7 +24,7 @@
<td v-if="!expense.confirmed">
<form v-on:submit.prevent="conf(expense)" method="POST" :action="'expense/' + expense.id + '/confirm/'">
{% csrf_token %}
<button class="theme-color btn-color" v-on:click="conf(expense, $event)">Bekräfta</button>
<button class="theme-color btn-color" v-on:click="conf(expense, $event)">Bekräfta giltighet</button>
</form>
</td>
<td v-else>
Expand All @@ -37,7 +37,7 @@
<td v-text="expense.amount + ' kr'"></td>
</tr>
</table>
<p v-else><b>Det finns inga icke-godkända kvitton i pärmen. Bra jobbat!</b></p>
<p v-else><b>Det finns inga icke-godkända kvitton. Bra jobbat!</b></p>
</div>

<script type="text/javascript">
Expand Down
2 changes: 1 addition & 1 deletion templates/admin/main.html
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ <h3>Utlägg och fakturor</h3>
{% if user.profile.may_view_confirm %}
<li>
<a href="{% url 'admin-confirm' %}">
Bekräfta i pärmen
Bekräfta kvittons giltighet
{% if counts.confirm > 0 %}
<span class="notification">{{ counts.confirm }}</span>
{% endif %}
Expand Down
8 changes: 3 additions & 5 deletions templates/expenses/confirmation.html
Original file line number Diff line number Diff line change
@@ -1,17 +1,15 @@
{% extends "../detail.html" %}
{% load widget_tweaks %}

{% block title %}{% if expense.is_digital %}Digitalt kvitto uppladdat{% else %}Sätt in kvittot i pärmen{% endif %}{% endblock %}
{% block title %}{% if expense.is_digital %}Digitalt kvitto uppladdat{% else %}Spara kvittot{% endif %}{% endblock %}

{% block full_content %}
<div>
{% if expense.is_digital %}
<p>Eftersom du markerade kvittot som digitalt behöver du inte sätta in det i pärmen.</p>
<p>Eftersom du markerade kvittot som digitalt behöver du inte spara kvittot förutom på Cashflow.</p>
<p>Om detta var fel - kontakta sektionens kassör, annars riskerar du att kvittot inte blir utbetalt.</p>
{% else %}
<p>Sätt in ditt kvitto i kvittopärmen i META på en <u><a href="/static/kvittomall.pdf">kvittomall</a></u>.</p>
<p>Märk kvittomallen med id:t <b>{{ expense.pk }}</b>. Du hittar det på din sida igen om du glömmer bort det.</p>
<p>Kvittopärmen <b>ska</b> vara sorterad efter Cashflow-ID i stigande ordning. Om du sätter in ditt kvitto fel riskerar du att det inte blir utbetalt.</p>
<p>Spara ditt kvitto ifall att bilden inte är tillräckligt bra eller att något behövs lämnas tillbaka.</p>
{% endif %}
<a href="{% url "expenses-show" expense.pk %}" class="button primary-action btn-color theme-color" style="color:#fff">Gå vidare</a>
</div>
Expand Down
11 changes: 5 additions & 6 deletions templates/expenses/show.html
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ <h2>Information</h2>
</td>
</tr>
<tr>
<th>Bekräftat i pärmen:</th>
<th>Giltighet bekräftad:</th>
<td v-if="expense.confirmed">
[[expense.confirmed_by]] ([[expense.confirmed_date]])
{% if user.profile.may_unconfirm %}
Expand Down Expand Up @@ -212,7 +212,6 @@ <h2>Kommentarer</h2>
<li class="alternatives">
<h3>Eller skicka en färdig kommentar:</h3>
<ul>
<li><a href="#" v-on:click="send">Jag kan inte hitta ditt utlägg i pärmen. Har du satt in det?</a></li>
<li><a href="#" v-on:click="send">Jag attesterar också detta.</a></li>
<li><a href="#" v-on:click="send">Det här har jag inte godkänt.</a></li>
<li><a href="#" v-on:click="send">Du har angivit större belopp än vad som står på kvittot.</a></li>
Expand All @@ -237,9 +236,9 @@ <h3>Eller skicka en färdig kommentar:</h3>
confirmed: true,
confirmed_by: "{{ expense.confirmed_by.get_full_name }}",
confirmed_date: "{{ expense.confirmed_at }}",
btn_message: "Avbekräfta",
btn_message: "Ta bort bekräftelse",
{% else %}
btn_message: "Bekräfta",
btn_message: "Godkänn bekräftelse",
confirmed: false,
{%endif%}
},
Expand Down Expand Up @@ -270,7 +269,7 @@ <h3>Eller skicka en färdig kommentar:</h3>
this.expense.confirmed = true;
this.expense.confirmed_date = (new Date()).toLocaleDateString('sv-SE', {year: 'numeric', month: 'long', day: 'numeric' })
this.expense.confirmed_by = this.user.full_name;
this.expense.btn_message = "Avbekräfta"
this.expense.btn_message = "Ta bort bekräftelse"
} else {
this.expense.btn_message = "Error! Försök igen"
}
Expand All @@ -293,7 +292,7 @@ <h3>Eller skicka en färdig kommentar:</h3>
this.expense.confirmed = false;
this.expense.confirmed_date = null;
this.expense.confirmed_by = null;
this.expense.btn_message = "Bekräfta"
this.expense.btn_message = "Godkänn bekräftelse"
} else {
this.expense.btn_message = "Error! Försök igen"
}
Expand Down
9 changes: 3 additions & 6 deletions templates/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@
{% comment "For later versions" %}
<ul class="info">
<li>Du har <a href="" v-tooltip="{ content: 'Muta möte<br>Teambuilding<br>Ledslingor' }">8 kvitton</a> du kan <a href="">attestera</a>.</li>
<li>Det finns <a href="">5 kvitton</a> att <a href="">bekräfta i kvittopärmen</a>.</li>
<li>Det finns <a href="">5 kvitton</a> att <a href="">bekräfta</a>.</li>
<li><a href="">3 personer</a> väntar på <a href="">utbetalning</a>.</li>
</ul>
{% endcomment %}
Expand All @@ -44,18 +44,15 @@
Därefter är det fritt fram att registrera kvitton i systemet.
</p>

<p>I Cashflow laddar du upp kvitton för dina utlägg digitalt. Detta för att förenkla sektionens ekonomiska arbete. Tyvärr, på grund av hur lagstiftningen kring bokföring ser ut, behöver man fortfarande lämna in kvittot fysiskt i kvittopärmen som finns i META.</p>
<p>I Cashflow laddar du upp kvitton för dina utlägg digitalt. Detta för att förenkla sektionens ekonomiska arbete. Kvittot bör fortfarande sparas ifall någonting skulle behövas lämnas tillbaka eller liknande.</p>

<p>När du laddat upp ett kvitto kan du följa kvittot från din användarsida och se när det är godkänt samt när det är utbetalat. På så sätt kan du enkelt jämföra dina bankhändelser med Cashflow och vara säker på att du fått tillbaka allt.</p>

<h2>Hur använder jag Cashflow och vad händer med kvittot?</h2>

<ol>
<li><b>Registrering.</b> Du som köpt något åt sektionen registrerar ditt kvitto här i Cashflow</li>
<li><b>Kvittopärmen.</b> Du sätter därefter in kvittot på en <u><a href="/static/kvittomall.pdf">sån här
mall</a></u> i METAs kvittopärm
</li>
<li><b>Kvittokontroll.</b> Sektionens kassör kontrollerar att kvittot är i kvittopärmen</li>
<li><b>Kvittobekräftelse.</b> Sektionens kassör bekräftar att kvittot uppfyller kravet för ett kvitto samt ser att det inscannade kvitto är av god kvalitet</li>
<li><b>Attestering.</b> Din nämndordförande, kassören eller ekonomiansvarig godkänner ditt kvitto.</li>
<li><b>Utbetalning.</b> Din nämndordförande, kassören eller ekonomiansvarig betalar ut kvittot. Flera kvitton
kan betalas ut samtidigt.
Expand Down

0 comments on commit e222dbc

Please sign in to comment.