Api serializers

[SalmanAsh]

This change is 

[SKairinos]

Reviewed 5 of 24 files at r1, 19 of 19 files at r2, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @SalmanAsh)

---

api/serializers/agreement_signature.py line 17 at r2 (raw file):

    class Meta:
        model = AgreementSignature
        fields = "__all__"

all is dangerous because if there were some fields that contained sensitive information that should not be returned to the front end, they would be returned. It's safer to manually specify the fields you want to return.

fields = ["id", "contributor", "agreement_id", "signed_at"]

---

api/serializers/contributor.py line 16 at r2 (raw file):

    class Meta:
        model = Contributor
        fields = "__all__"

all is dangerous because if there were some fields that contained sensitive information that should not be returned to the front end, they would be returned. It's safer to manually specify the fields you want to return.

fields = ["id", "email", "name", "location", "html_url", "avatar_url"]

[SKairinos]

Reviewed 2 of 2 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @SalmanAsh)