codefiction / codefiction-tech-lambda Goto Github PK

UI Component'inin testi var, api'nin yok.

#2 'ında template.yml'da yapılan değişikliklerle ezilen endpoint'lerin düzeltilmesi ve her request'i function'a yönlendirecek şekilde yapılandırılması.

/podcasts
/podcasts/search

Sizce de daha şık olmaz mı?

Arkadaşlar arama fonksiyonu ile ilgili gözüme takılanları yazmak istedim. Feature planlanırken aranızda olamadığım için özürlerimi iletirim öncelikle. Ve eğer aşağıda yazdıklarım zaten tartışıldıysa direkt issue'yu siliniz plz.

• Arama fonksiyonunu önyüzde yapamaz mıyız? Neden bunun için bir daha lambda fonksiyonu çalıştırıyoruz. Dom üzerinde arama yapmak daha pratik olur ve type as you go bir arama implement edebiliriz.
• Bu sayede içinde ara butonunu kaldırabiliriz. input alanındaki placeholder'a da güzel bişiler yazarız, butona ihtiyaç kalmaz.
• Şu anda arama yaptıktan sonra aramayı temizlemek mümkün değil, boş bir sayfa kalıyor karşımda. Tüm bölümleri görmek için refresh etmek gerekiyor. Oysa beklediğim hareket inputu temizleyince tüm podcastlerin tekrar önüme gelmesi.

• Bir de şu haliyle arama yaparken bi loading indicator falan çıkmaması, kullanıcıya hiçbir feedback vermiyoruz.

Sizler ne düşünüyorsunuz?

Podcast'leri getiren ve search eden endpoint'ler /podcasts altına taşındı. Böyle hem AWS Api Gateway de yapılan "greedy path parameter" #3 değişikliğinin çalışması sağlanacak hem de api başka amaçlarla da kullanılabilecek.