Should also check UBI pool if wallet can claim

[sirpy]

1. Currently the claim method doesnt call the UBI pool canClaim method.
   This can result in a claim TX doing nothing, just wasting gas

2. There should be a single method for a developer to check the wallet claim status

• not whitelisted - developer knows he still need to show the claim button that will redirect to get verified (or show a custom message)
• can claim (entitlement > 0) - developer knows to show claim button
• already claimed - developer knows to show timer or custom message

3. consider have nextClaimTime to return 0 if wallet checkEntitlement > 0.

GoodSDKs/packages/citizen-sdk/src/sdks/viem-claim-sdk.ts

Line 198 in 62a0eeb

const canClaim = await this.checkBalanceWithRetry()

[Oliverpt-1]

Hey! @sirpy @L03TJ3 Can I please be assigned this issue?

[sirpy]

@Oliverpt-1 assigned. GL.
https://ubi.gd/GoodBuilders

We have office hours and available on TG/Discord if you have any questions or blockers.

[sirpy]

@L03TJ3 fYI

[Oliverpt-1]

Great thanks @sirpy ! Just put up a PR for it.

[vortex-hue]

Hi Guys, is this completed or still open for assignment?

[L03TJ3]

In the timeline you see a PR has been merged and accepted.
We leave the ticket open because it has not been released to prod yet.

Going forward, I will remove the scoutgame label

[vortex-hue]

alright, thanks for the clarification lewis.