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

Added some missing admin registrations. #300

Merged
merged 3 commits into from
Dec 13, 2016
Merged

Added some missing admin registrations. #300

merged 3 commits into from
Dec 13, 2016

Conversation

freakboy3742
Copy link
Contributor

This adds admin registrations for Card and BitcoinReceiver (as inlines on Customer), and for TransferChargeFee (as an inline on Transfer).

This also enforced extra=0 and max_num=0 on inlines; this makes the admin representation much more terse (as there are no blank extra lines in the default rendering), and is as close as you can easily get to "readonly" (as you can't add new lines).

felixxm and others added 2 commits November 29, 2016 12:02
Also disabled empty lines and the Add another button on inlines.
@coveralls
Copy link

coveralls commented Dec 13, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling ac4d7a6 on jambonsw:card-in-admin into aaa4516 on pinax:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling ac4d7a6 on jambonsw:card-in-admin into aaa4516 on pinax:master.

@coveralls
Copy link

coveralls commented Dec 13, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 46d158b on jambonsw:card-in-admin into aaa4516 on pinax:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 46d158b on jambonsw:card-in-admin into aaa4516 on pinax:master.

@freakboy3742
Copy link
Contributor Author

This patch includes the content of #297, as the test suite won't pass without it.

@coveralls
Copy link

coveralls commented Dec 13, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 8aa80ac on jambonsw:card-in-admin into aaa4516 on pinax:master.

@paltman
Copy link
Member

paltman commented Dec 13, 2016

Thanks @freakboy3742!

@paltman paltman merged commit 8333755 into pinax:master Dec 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants