onflow / onflow/flow

block_height is missing from access openapi schema

Open
#1,024 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug Feedback
Dominant language
Go
Stars
361
Forks
159
PR merge metrics
No merged PRs in 30d

Description

Problem

flow-go-sdk needs to be updated to include block_height in the transaction result but it is missing from the access openapi schema.

Steps to Reproduce

N/A

Acceptance Criteria

None

Context

flow-go-sdk builds its json models using swagger-codegen generate -l go -i https://raw.githubusercontent.com/onflow/flow/master/openapi/access.yaml -D packageName=models,modelDocs=false,models -o access/http/models;

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the access OpenAPI schema referenced by the swagger-codegen command, openapi/access.yaml, and locate the transaction result definition. Check how flow-go-sdk generates its Go models from that schema. Done means the transaction result schema includes block_height and the generated model can represent it.

Written by the indexing model from the issue text.

Assessment

Tech stack
go, openapi
Domain
api, blockchain
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.