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.

Sign up for GitHub

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

Jump to bottom

Add check for NftReadableKVState if spender is missing #9792

Closed
Tracked by #8828
nickeynikolovv opened this issue Nov 21, 2024 · 0 comments · Fixed by #9790
Closed
Tracked by #8828

Add check for NftReadableKVState if spender is missing #9792

nickeynikolovv opened this issue Nov 21, 2024 · 0 comments · Fixed by #9790
Assignees
@nickeynikolovv
Labels
enhancement Type: New feature web3 Area: Web3 API
Milestone
0.119.0

Comments

@nickeynikolovv
Copy link
Contributor

nickeynikolovv commented Nov 21, 2024 •
edited
Loading

With the current implementation of NftReadableKVState if spender is not set for NFT it returns null. We should add a check if spender is not set
@nickeynikolovv nickeynikolovv converted this from a draft issue Nov 21, 2024
@nickeynikolovv nickeynikolovv self-assigned this Nov 21, 2024
@nickeynikolovv nickeynikolovv added enhancement Type: New feature web3 Area: Web3 API labels Nov 21, 2024
@nickeynikolovv nickeynikolovv mentioned this issue Nov 21, 2024
Merged
2 tasks
@steven-sheehy steven-sheehy mentioned this issue Nov 25, 2024
Open
@steven-sheehy steven-sheehy modified the milestones: 0.120.0, 0.119.0 Nov 25, 2024
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Mirror Node Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nickeynikolovv nickeynikolovv

Labels
enhancement Type: New feature web3 Area: Web3 API
Projects
Mirror Node
Status: ✅ Done
Milestone
0.119.0
Development

Successfully merging a pull request may close this issue.

Add check for NftReadableKVState if spender is missing hashgraph/hedera-mirror-node
2 participants
@steven-sheehy @nickeynikolovv