-
Notifications
You must be signed in to change notification settings - Fork 9
Conversation
# Conflicts: # core/filters.py # core/models.py # core/rest.py
@@ -45,4 +45,3 @@ class UserAdmin(_UserAdmin): | |||
admin.site.register(User, UserAdmin) | |||
admin.site.register(Semester) | |||
admin.site.register(Card) | |||
admin.site.register(NfcCard) |
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.
Er det en spesiell grunn til at du fjerner det fra admin?
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.
Skjedde en feil når jeg merga den reverta branchen med den branchen som var her, skal få fiksa det.
Forstår jeg det korrekt at man beholdt løsningen med to forskjellige modeller for kortene? |
@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! |
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 |
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.
VoucherWallet
er ikke importert
Vil vel si at dette er klar til å merges, noen kommentarer? |
Jeg tror koden er klar for å merges, og om det er noe feil så fikser vi det bare senere. |
Har fått lagt inn støtte for kaffe bonger.