-
Notifications
You must be signed in to change notification settings - Fork 30
Utkuaysev homework 3 #85
base: main
Are you sure you want to change the base?
Conversation
upy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
field name'leri get_all_base_abstract_model_attrs bu sekilde yapmamizin motivasyonu ne idi?
| from django.shortcuts import render | ||
|
|
||
| # Create your views here. | ||
| # from rest_framework import viewsets |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bu dosyanin tamamini neden yorum yaptik?
ecommerce/orders/serializers.py
Outdated
| class BillingAddressSerializer(serializers.ModelSerializer): | ||
| class Meta: | ||
| model = BillingAddress | ||
| fields = get_all_billing_address_attrs() + get_all_base_abstract_model_attrs() + "id" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
burada hata alir miyiz?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Teşekkür ederim hocam düzelttim.
reusable olması ve yeni bir field eklendiğinde bütün field'leri çeken yerleri güncellemek zorunda kalmamak. |
anliyorum, yapilabilir ama sahsen okunabilirligi azalttigi hissiyatim var veya hic boyle bir kullanim tercihi gormedigimden bana oyle geliyor olabilir. |
view and manager pages are missing. will updated later. filters,serializers are done.