search

Ardakilic / laravel-mutlucell-sms

Laravel için Mutlucell SMS Kütüphanesi
MIT License
45 stars 11 forks source link

Gönderimden önce varsayılan config değerlerini override etme #5

Closed shibby closed 7 years ago

shibby commented 7 years ago

Selam Arda,

Bizim kurguladığımız bir sistemde, Mutlucell kullanıcı adı-parolası env ya da config dosyasından değil de, veritabanından geliyor. Dolayısıyla, her gönderimden önce kullanıcı adı/parola/senderId gibi bilgileri tekrar göndermemiz gerekiyor.

Bunu nasıl yapabiliriz? En doğru yolu nedir? Tartıştıktan sonra ben PR gönderirim. Eğer böyle bir feature'a gerek yok dersen, yine de yöntemi tartışalım. Forklayıp kullanırım :)

Ardakilic commented 7 years ago

Selam,

Mevcut yapıyı bozmadan en temiz yolu send* metodlarına opsiyonel yeni bir parametre eklenip bu parametrede k.adı, parola, originatör vs. değerleri array olarak metoda yollamak gibi duruyor.

Ne dersin ?

shibby commented 7 years ago

Ben setCredentials($params) gibi bir fonksiyon eklemeyi dusunmustum. Hem construct'ta bu fonksiyon kullanarak parametreleri set ederiz. Hem de istersek ::send() metodundan once degistirmek icin kullaniriz.

\Mutlucell::setCredentials()->send()

On January 11, 2017 11:20:45 AM GMT+03:00, "Arda Kılıçdağı" notifications@github.com wrote:

Selam,

Mevcut yapıyı bozmadan en temiz yolu send* metodlarına opsiyonel yeni bir parametre eklenip bu parametrede k.adı, parola, originatör vs. değerleri array olarak metoda yollamak gibi duruyor.

Ne dersin ?

-- You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub: https://github.com/Ardakilic/laravel-mutlucell-sms/issues/5#issuecomment-271806242

-- Sent from my Android device with K-9 Mail. Please excuse my brevity.

Ardakilic commented 7 years ago

@shibby Hm o da olabilir, istersen bir dene, PR yolla, üstünden bakalım beraber 👍 . Ya asıl kaygım mevcut metodların kullanımı değişmesin.

Ardakilic commented 7 years ago

@shibby biraz yoğunluktan geç geri döndüm özür

Bi göz gezdirebilir misin şu commit'e ? https://github.com/Ardakilic/laravel-mutlucell-sms/commit/6d432d3a527e354361dd11d5a88d196e843be3fe

Eklemek / değişsin dediğin bir şey var mı ? Yoksa release çıkarayım.

shibby commented 7 years ago

@Ardakilic np, daha iyi olmuş tabii ki böyle :+1: döküman da iyi olmuş, eline sağlık.

bir de, yazdığın pakete feature ekleyenlerin pull requestlerini kabul etmeden önce döküman yazmalarını, -varsa- test yazmalarını isteyebilirsin. Senin üzerindeki yoğunluğu alacaktır :)

Ardakilic commented 7 years ago

@shibby Dediğin doğru, Github issue template'lerde pull request placeholder metinleri var, bi ara üşenmezsem (😆) onu doldurmayı düşünüyorum ben de.

L4 ve L5 için 2 sürüm geçtim, biri 2.4.0 , diğeri 1.3.0 👍