Field name of reversed relations

[g-as]

Hi,

django field name is used to determine the field name for the corresponding strawberry type.

This is obviously fine, but in the case of reversed relations, the related_name would be more appropriate IMO.

Something like that would do the trick:

from django.db.models.fields.reverse_related import ForeignObjectRel

def get_model_fields(cls, model, fields, types, is_input, partial):

    [...]

    if isinstance(field, ForeignObjectRel)
        field_name = field.get_accessor_name()
    else:
        field_name = field.name

    model_fields.append((field_name, field_type, field_value))

I'll draft a PR tomorrow.

Upvote & Fund

• We're using Polar.sh so you can upvote and help fund this issue.
• We receive the funding once the issue is completed & confirmed by you.
• Thank you in advance for helping prioritize & fund our backlog.

[la4de]

@g-as, thank you for feedback.

We are in the middle of publishing enhanced API where all fields would be defined in class body as discussed in #20.

This internal function is refactored there and I definitely want to take this into account there.

[g-as]

A'ight, I'll go take a look.

[la4de]

This should work now with code in features/class-api branch. Feel free to try.

[g-as]

You rock!

[la4de]

See pull request #30