Issue #5: Auth, Social Auth, Profile Sayfalari#48
Conversation
bahattincinic
changed the title
|
Eline sağlık Bahattin :) Henüz bakmadım koda, CSS kısmına ben de el atarım. Tişört iyiymiş :P People modelini custom User modeli altında birleştirmek iyi fikir. Prod veritabanından o tabloyu dump edebilirim, onu da data migration yaparken kullanabiliriz. |
|
Tamamdir boyle daha cok icime sinicek. Ben custom user model yapmayiz koydugum link modeline koyariz diye dusunmustum ama. Denormalize olsun Useri ezelim github, twitter blog linklerini oyle koyalim dersen oda olur. |
|
Buna henüz bakamadım Bahattin kusuruma bakma. Benden önce bakan olmazsa 2-3 hafta sonra bakabileceğim ancak :( |
|
Ben ozaman insiyatif kullanarak ilerliyim en iyisi onun uzerine konusalim. |
|
👍 |
|
@berkerpeksag Bu pr a @umitdincel devam edicem ile. Umit site icin yeni tasarim yaptirtmis sizlede konusmus zaten. Onu da benim branch ustunden cikip yapicaz. |
|
Ben yerinizde olsam bu branch master'a girmeden tasarımı giydirme işine girmezdim. Tasarımda olup sitenin şu anki halinde olmayan başka şeyler de vardı diye hatırlıyorum. Önce o maddeleri halledip en son tasarımı adapte etmek daha kolay olabilir. |
|
Tamamdir ozaman oyle yapalim. Ben buradaki eksikleri hallediyem sonrasinda tasarim ile ilgili seylere baslayalim. |
|
Bir de tasarıma Meetup'dan gelen etkinlikleri de entegre etmek gerekecek. Bunları ayrı bir issue'da konuşuruz :) |
|
etkinlik icin hem meetup dan gelenleri tutmak hemde biz kendimiz de tutmak yerine Birini sadece kullansak ? |
|
Pardon bizdeki o blog postu du 😄 |
|
@berkerpeksag Bu PR i close ediyim mi ? Yoksa cleanup branch ini merge edip bitiriyim mi bunu ? |
2 participants
People ile kullanici modelini view/template de merge edicektim ama cok hosuma gitmedi. acaba direk modeli ucursak da User tablosuna initial olarak koysak mi veya datamigration @berkerpeksag
Yukaridaki 2 madde kaldi. Onlari yapiyim oyle merge ederiz. Bu arada yorum varsa da hemde tartismis oluruz.
Yaptigim profil sayfasi su. Baya kotu oldu inline css falan yazdim. Fatihin ustunden gecmesi lazim
