Skip to content
This repository has been archived by the owner on Nov 14, 2024. It is now read-only.

Kaffe bonger #38

Merged
merged 18 commits into from
Jul 11, 2016
Merged

Kaffe bonger #38

merged 18 commits into from
Jul 11, 2016

Conversation

nicolaei
Copy link
Contributor

@nicolaei nicolaei commented Jul 5, 2016

Har fått lagt inn støtte for kaffe bonger.

@nicolaei nicolaei mentioned this pull request Jul 5, 2016
@@ -45,4 +45,3 @@ class UserAdmin(_UserAdmin):
admin.site.register(User, UserAdmin)
admin.site.register(Semester)
admin.site.register(Card)
admin.site.register(NfcCard)
Copy link
Contributor

Choose a reason for hiding this comment

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

Er det en spesiell grunn til at du fjerner det fra admin?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Skjedde en feil når jeg merga den reverta branchen med den branchen som var her, skal få fiksa det.

@henrist
Copy link
Member

henrist commented Jul 6, 2016

Forstår jeg det korrekt at man beholdt løsningen med to forskjellige modeller for kortene?

@nicolaei
Copy link
Contributor Author

nicolaei commented Jul 6, 2016

@henrist Ja, men ser ut som det skjedde noe rart når jeg merga med branchen her siden jeg hadde reverta på min side.

@henrist
Copy link
Member

henrist commented Jul 6, 2016

@henrist Ja, men ser ut som det skjedde noe rart når jeg merga med branchen her siden jeg hadde reverta på min side.

Bare for å nevne det så har dere i og for seg overtalt meg til å trekke UiO kort helt ut av bongsystemet, som vi var inne på i en annen kommentar her. Bra dere har gode innspill!

nicolaei added 4 commits July 7, 2016 14:38
Yeh, I sorta forgot that you should be able to use them as well...
And this is why I should always double check the code I commit...
@@ -11,8 +12,8 @@
from decimal import Decimal

from voucher.serializers import *
from voucher.models import Wallet, VoucherRegisterLog, UseLog, VoucherUseLog, CoffeeRegisterLog, CoffeeUseLog
Copy link
Member

Choose a reason for hiding this comment

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

VoucherWallet er ikke importert

@nicolaei
Copy link
Contributor Author

Vil vel si at dette er klar til å merges, noen kommentarer?

@matsasc
Copy link
Contributor

matsasc commented Jul 11, 2016

Jeg tror koden er klar for å merges, og om det er noe feil så fikser vi det bare senere.

@nicolaei nicolaei merged commit 6179627 into master Jul 11, 2016
@nicolaei nicolaei deleted the nfc-system branch July 11, 2016 16:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants