Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rethink repr usage #18

Open
jarekwg opened this issue Jun 6, 2018 · 0 comments
Open

Rethink repr usage #18

jarekwg opened this issue Jun 6, 2018 · 0 comments

Comments

@jarekwg
Copy link
Member

jarekwg commented Jun 6, 2018

I've written the __repr__ methods in a way that makes it super quick to know what methods are available on a manager. However, when looking at a list of companyfiles, you end up getting something like this:

[CompanyFile:
     contacts
     general_ledger
     inventory
     invoices
     purchase_bills
     purchase_orders,
CompanyFile:
     contacts
     general_ledger
     inventory
     invoices
     purchase_bills
     purchase_orders]

This obviously (a) looks silly, and (b) provides no insight as to what each companyfile is. Should at the very least include the name and uid of the companyfile, and also consider moving the method list to a separate methods() function rather than having the repr list available endpoints.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant