hoodrunio / explorer-backend

Backend app for Cosmos ecosystem explorer.
https://backend.testnet.run/
5 stars 1 forks source link

Generic responses #20

Closed amorfc closed 1 year ago

amorfc commented 1 year ago

Success responsların hepsi

{data:<T>}

Şeklinde dönülecek şekilde tamamlandı. İsteğe bağlı istersek yanında başka yapılarda dönebiliriz ona müsait.

Errorların hepside aynı şekilde

{error:String}

Şeklinde dönülecek şekilde tamamlandı. İsteğe bağlı içede generic mesaj enumları oluşturuldu. Bu yapıyı bu branchte test edip onaylarsanız diğer enpointlere eklenmiş haliyle göndereceğim.

- Test için tokenomics enpointleri implement ettim kullanım. - Hatayı test etmek için chain yanlış girerek istek atabilirsiniz.

Herkesi etkileyecek bir yapı olduğu için review ve onayınız önemli diye düşünüyorum 🤩 @mehmetuken @dvtkrlbs localde hızlı bir göz atabilir misiniz test için teşekkürler.

dvtkrlbs commented 1 year ago

Tam aklimda olan yapi bu deildi ama looks good to me. Error handling icin thiserror gibi bir crate kullanabiliriz cok repetative zaten error typelari olusturmak. Error Responseini ekstradan string olarak tanimlamaya gerek yok bence. Direkt bi middleware yazip error tipini json olarak dondurebiliriz frontende.

amorfc commented 1 year ago

Tam aklimda olan yapi bu deildi ama looks good to me. Error handling icin thiserror gibi bir crate kullanabiliriz cok repetative zaten error typelari olusturmak. Error Responseini ekstradan string olarak tanimlamaya gerek yok bence. Direkt bi middleware yazip error tipini json olarak dondurebiliriz frontende.

erroru json olarak doneceksek bu durumda fe her errorun içeriğini duruma göre parse etmek zorunda kalacak en azından mvpden sonrasına bırakabiliriz bu kısmı diye düşünüyorum zaten yapı değiştirilmeye müsait ne dersiniz @dvtkrlbs @Errorist79

mehmetuken commented 1 year ago

test edip anlayacak kadar rust' a hakim değilim ama bana da yeterli ve güzel gözüktü eline sağlık. teşekkürler.