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

Height param is not processed querying custom module/list #3435

Closed
RaulBernal opened this issue Feb 15, 2023 · 2 comments · Fixed by #3441
Closed

Height param is not processed querying custom module/list #3435

RaulBernal opened this issue Feb 15, 2023 · 2 comments · Fixed by #3441
Labels
type:bug Something isn't working

Comments

@RaulBernal
Copy link
Contributor

RaulBernal commented Feb 15, 2023

Describe the bug
After scaffold a custom module/list you can't query results by height.

To reproduce
Steps to reproduce the behavior:

  1. Scaffold a module/list
    In example:
    ignite scaffold list bitcannaid bcnaid address
  2. Start the blockchain
  3. Insert one value at block X
    In example:
    bcnad tx bcna create-bitcannaid JoeID bcna33333joe --from validator
  4. Update that value at block X+10 in example
    bcnad tx bcna update-bitcannaid 0 JoeID bcna33333 --from validator
  5. List all the fields or the field with ID 0 at height X and X+10 to see the difference. It doesn't works

I've fixed the CLI query side:
BitCannaGlobal/bcna@2e90dc3

But I want to know how to update the LCD query because it doesn't handle the ?height=xxxx param
In example: curl http://localhost:1317/BitCannaGlobal/bcna/bcna/bitcannaid/0?height=100

What version are you using?
v0.25.x & v0.26.x

@RaulBernal
Copy link
Contributor Author

Hi @ilgooz @Pantani the issue remains at LCD queries as I commented here #3441 (comment)

@RaulBernal
Copy link
Contributor Author

I guess something is missing with the context here, in example: @Pantani
https://github.com/BitCannaGlobal/bcna/blob/main/x/bcna/keeper/bitcannaid.go#L63

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant