feat: Model/schema additions for admin billing info

[mrsaicharan1]

Fixes #6097

Checklist

• I have read the Contribution & Best practices Guide and my PR follows them.
• My branch is up-to-date with the Upstream development branch.
• The unit tests pass locally with my changes
• I have added tests that prove my fix is effective or that my feature works
• I have added necessary documentation (if appropriate)

• All the functions created/modified in this PR contain relevant docstrings.

Changes proposed in this pull request:

Adds admin invoice details

• schema additions
• model additions
• migration file

[codecov]

Codecov Report

Merging #6098 into development will increase coverage by 0.06%.
The diff coverage is 100%.

@@               Coverage Diff               @@
##           development    #6098      +/-   ##
===============================================
+ Coverage        66.16%   66.23%   +0.06%     
===============================================
  Files              285      285              
  Lines            14212    14241      +29     
===============================================
+ Hits              9403     9432      +29     
  Misses            4809     4809

Impacted Files	Coverage Δ
app/models/setting.py	90.22% <100%> (+1.19%)	⬆️
app/api/schema/settings.py	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5ae00ff...4288001. Read the comment docs.

[mrsaicharan1]

@shreyanshdwivedi @uds5501 Please review.

[mrsaicharan1]

@shreyanshdwivedi @uds5501 @iamareebjamal Removed the server defaults. Please have a look.

[mrsaicharan1]

@iamareebjamal @CosmicCoder96 @uds5501 @shreyanshdwivedi Made the requested changes. Please review.

[mrsaicharan1]

Okay, might be something on my side. Will check

…

On Wed, 26 Jun 2019 at 9:05 PM, Areeb Jamal ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In app/api/schema/settings.py <#6098 (comment)> : > @@ -196,3 +196,16 @@ class Meta: smtp_password = fields.Str(allow_none=True) smtp_port = fields.Integer(allow_none=True) smtp_encryption = fields.Str(allow_none=True) # Can be tls, ssl, none + + # Admin Invoice Details + admin_billing_contact_name = fields.Str(allow_none=True) + admin_billing_phone = fields.Str(allow_none=True) + admin_billing_email = fields.Email(allow_none=True) + admin_billing_country = fields.Str(allow_none=True) + admin_billing_tax_info = fields.Str(allow_none=True) + admin_company = fields.Str(allow_none=True) + admin_billing_company_address = fields.Str(allow_none=True) + admin_billing_company_city = fields.Str(allow_none=True) + admin_billing_zip = fields.Str(allow_none=True) + admin_billing_additional_info = fields.Str(allow_none=True) + admin_billing_company_comments = fields.Str(allow_none=True) @mrsaicharan1 <https://github.com/mrsaicharan1> It'll automatically show outdated once you have done the change — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#6098?email_source=notifications&email_token=AGAHUWZNKC2MZS3NSLUXCUTP4OED7A5CNFSM4H2X5ZT2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB4XHAKI#discussion_r297734345>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AGAHUWYXX2XEQJVLIK5QUD3P4OED7ANCNFSM4H2X5ZTQ> .

[mrsaicharan1]

@CosmicCoder96 I've made the changes. Please review.

[mrsaicharan1]

@shreyanshdwivedi @uds5501 Please do a peer review.

[iamareebjamal]

There's too much repetition. Open an issue to use dataclasses in future:
https://realpython.com/python-data-classes/