My invoices are created but appear nowhere
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Maintainer
Yes I noticed this also. The invoice is created, the project is set properly, but
projects/1100/customerInvoices
returns an empty container.@calummackervoy any idea of why it does not work?
Edited by Matthieu Fesselier Collapse replies Does it exist in the admin? If so then it's a serialization issue
Edited by Calum Mackervoy- Maintainer
Yes it does
- Maintainer
Actually, since today, I cannot see
customerInvoices
andfreelanceInvoices
here: https://api.test1.startinblox.com/projects/1100/Has it been removed from test1?
Not for me? I get back all 3 of the customer invoices?
None of them were attached to the customer
John Doe
when created (bug?) but otherwise they look fine?- Maintainer
Weird, now they appear. But the containers are empty: https://api.test1.startinblox.com/projects/1100/
None of them were attached to the customer
John Doe
when created (bug?)Yes I think I did not add this in the model
but otherwise they look fine?
Yes they are created and attached, I don't know why they are not in the project document
Okay I'm fairly certain this issue is to do with permissions - there's a known issue that
LDPPermissions
will analyse the child permissions with the parent's permission class, passing the child objectThis is no longer the case as of the permissions refactor, so I'd like to debug this properly after that is merged. I'll need to open an MR to update
InvoicePermissions
, but when it's merged it will be a lot easier to spot permissions bugs- Maintainer
Indeed, I just wrote something very simple which might be totally wrong for the
InvoicePermissions
, feel free to review it if you want :) It looks fine I just meant that in the refactor
user_permissions
is being abolished - I meant when the DjangoLDP MR is merged- Maintainer
Ok got it, so I need to wait for the merge of DjangoLDP for this to work, right?
I think it's possible to fix now but I'd like to wait until then so I don't need to fix it twice
- Maintainer
No problem, let me know when this is merged ;)
- Matthieu Fesselier mentioned in issue #20 (closed)
mentioned in issue #20 (closed)
@balessan are you able to reproduce this bug? I tried but for me the invoice appears, and I can GET it when visiting the URL https://api.test1.startinblox.com/projects/1100/customerInvoices/
Collapse replies I've replicated with the other account !
- Calum Mackervoy mentioned in merge request djangoldp-packages/djangoldp-invoice!9 (merged)
mentioned in merge request djangoldp-packages/djangoldp-invoice!9 (merged)
- Calum Mackervoy mentioned in issue djangoldp-packages/djangoldp#341 (closed)
mentioned in issue djangoldp-packages/djangoldp#341 (closed)
Found another part of this issue: djangoldp-packages/djangoldp#341 (closed)
Worked around it for now by disabling the serializer cache on test1
- Calum Mackervoy mentioned in merge request djangoldp-packages/djangoldp-invoice!10 (merged)
mentioned in merge request djangoldp-packages/djangoldp-invoice!10 (merged)
- Matthieu Fesselier closed via merge request djangoldp-packages/djangoldp-invoice!10 (merged)
closed via merge request djangoldp-packages/djangoldp-invoice!10 (merged)