Content pfp
Content
@
0 reply
0 recast
0 reaction
Pau 🎩 ⛓️‍💥 pfp
Pau 🎩 ⛓️‍💥
@p4u
Hey, listen👂this week's new feature has been deployed at farcaster.vote Now you can create Farcaster gated polls 🗳️ for your Alfafrens channel subscribers!! 🚀 Follow @vocdoni to stay up to date with new developments.
2 replies
6 recasts
12 reactions
Pau 🎩 ⛓️‍💥 pfp
Pau 🎩 ⛓️‍💥
@p4u
We made a mistake using the Alfafrens new API, we did not consider pagination, lol 🫠 We are fixing it. Pools won't work properly until then. cc/ @taimonania
2 replies
0 recast
1 reaction
Pau 🎩 ⛓️‍💥 pfp
Pau 🎩 ⛓️‍💥
@p4u
So @vijay there is something wrong with the API. We are using this method https://docs.alfafrens.com/docs/alfafrens-api/get-channel-subscribers-and-stakes But it returns fewer subscribers than the ones shown by the Alfafrens UI. I.e for @taimonania it returns a total of 153 subs, while the UI says 375.
1 reply
0 recast
1 reaction
Pau 🎩 ⛓️‍💥 pfp
Pau 🎩 ⛓️‍💥
@p4u
We paginate with first=200&skip=0 then first=200&skip=200 then first=200&skip=400 and so on. Until "hasMore" is false. Which seems correct on our side. We take only the FID if "member.isSubscribed" is true. I hope it is the API wrong and not the Alfafrens UI showing a wrong number of subs 😘
1 reply
0 recast
0 reaction
Pau 🎩 ⛓️‍💥 pfp
Pau 🎩 ⛓️‍💥
@p4u
I think I found the issue @vijay, there is a hardcoded limit of 1000 for the skip parameter. So if the channel has more items, they cannot be fetch... { "message": "first and skip must be less than 1000", "code": "PAYLOAD_TOO_LARGE" }
3 replies
0 recast
0 reaction
vijay pfp
vijay
@vijay
@naxe replied below but lmk if that doesn’t fix it for you. DMs open
2 replies
0 recast
0 reaction
Pau 🎩 ⛓️‍💥 pfp
Pau 🎩 ⛓️‍💥
@p4u
@naxe I would suggest two changes: 1. A new query parameter "subscribed=true/false", if specified, the API returns only the members either subscribed or not (it is overkill to receive the full list always). 2. Remove the hardcoded limit for skip > 1000, else it is impossible to paginate the entire list.
1 reply
0 recast
3 reactions
Axe pfp
Axe
@naxe
the skip should not be limited this way, was a bug. Is fixed, trying to push a new version to prod. I agreed with the param, we can polish the results before send it back 🙏
1 reply
0 recast
3 reactions
Pau 🎩 ⛓️‍💥 pfp
Pau 🎩 ⛓️‍💥
@p4u
Ping me when the fix is deployed please.
1 reply
0 recast
1 reaction
Axe pfp
Axe
@naxe
doing the last reviews and pushing a new version, 30min and we are done =)
2 replies
0 recast
2 reactions
Axe pfp
Axe
@naxe
also I added you idea to show only subs or stake = true
1 reply
0 recast
0 reaction
Pau 🎩 ⛓️‍💥 pfp
Pau 🎩 ⛓️‍💥
@p4u
Great. Just tested it now... and the "hasMore" boolean has disappeared, so there is no way to know if there are more pages.
1 reply
0 recast
1 reaction