search

PragmaticMates / django-invoicing

Django app for invoicing
GNU General Public License v2.0
99 stars 36 forks source link

exporters: InvoiceXlsxListExporter.get_model #10

Closed matejhazala closed 1 year ago

matejhazala commented 1 year ago

added get_model method required by django-output permission widget

eriktelepovsky commented 1 year ago

I'll merge for now but it should be probably better to put the method into one of the mixins I think.

matejhazala commented 1 year ago

it is in filterexportermixin i just thought maybe didnt want to subclass that necessarily ant thi was only method needed to run, i can do it other way

On Sun, 15 Jan 2023 at 13:24, Erik Telepovský @.***> wrote:

Merged #10 https://github.com/PragmaticMates/django-invoicing/pull/10 into master.

— Reply to this email directly, view it on GitHub https://github.com/PragmaticMates/django-invoicing/pull/10#event-8229513588, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADS72R4V6HWKBBJLHGKNF23WSP3BNANCNFSM6AAAAAAT3PGBYM . You are receiving this because you authored the thread.Message ID: @.*** com>

eriktelepovsky commented 1 year ago

I'll look into it some day. It deserves better refactoring anyway. Thx.