Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chore: Change proposals endpoint #23

Open
Errorist79 opened this issue Jan 11, 2023 · 0 comments
Open

Chore: Change proposals endpoint #23

Errorist79 opened this issue Jan 11, 2023 · 0 comments
Assignees
Labels

Comments

@Errorist79
Copy link
Member

Errorist79 commented Jan 11, 2023

Cosmos sdk, iki farklı proposal endpointi sunar:
/cosmos/gov/v1beta1/proposals ve /cosmos/gov/v1/proposals

Mevcut yapıda kullandığımız endpoint: /cosmos/gov/v1beta1/proposals:
https://github.com/testnetrunn/explorer-backend/blob/main/src/fetch/proposals.rs#L21
https://github.com/testnetrunn/explorer-backend/blob/main/src/fetch/proposals.rs#L61
https://github.com/testnetrunn/explorer-backend/blob/main/src/fetch/proposals.rs#L113
https://github.com/testnetrunn/explorer-backend/blob/main/src/fetch/proposals.rs#L141
https://github.com/testnetrunn/explorer-backend/blob/main/src/fetch/proposals.rs#L152
https://github.com/testnetrunn/explorer-backend/blob/main/src/fetch/proposals.rs#L164
https://github.com/testnetrunn/explorer-backend/blob/main/src/fetch/proposals.rs#L191

Fetchlenen veri türlerinde de bazı farklılıklar var, örneğin ilk endpointte proposal_id değeri, ikinci endpointte yalnızca id olarak isimlendirilir. buna uygun şekilde verileri almamız gerekecek.

Örneklendirecek olursak, aynı proposalin iki farklı endpointten gelen veri türlerini karşlaştırabiliriz:

Mevcutta kullandığımız endpoint: https://evmos-api.polkachu.com/cosmos/gov/v1beta1/proposals/97
Kullanmayı düşündüğümüz endpoint: https://evmos-api.polkachu.com/cosmos/gov/v1/proposals/97

Birden fazla endpoint olduğundan, yani /cosmos/gov/v1beta1/proposals/tally, /cosmos/gov/v1beta1/proposals/proposal_id/votes... gibi türev endpointlerin de veri türlerini karşılaştırmak gerekebilir.

Güncelleme:

İlk endpoint, yani mevcut yapıda kullandığımız endpoint şu anda tüm ağlarda çalışıyor.
İkinci endpoint ise, yalnızca sdk sürümü 0.46.x olan sürümlerde çalışıyor. Bu yüzden sdk sürümü 0.46.x'ten küçük ise ilk yapıyı kullanmaya zorlayacağız, kısacası mevcut yapıyı tamamen silmiyoruz.

@Errorist79 Errorist79 pinned this issue Feb 6, 2023
@amorfc amorfc assigned amorfc and unassigned dvtkrlbs Feb 14, 2023
@0xluen 0xluen unpinned this issue Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants