Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

`liquidity` is a column with no definition

Đang mở
  • Có gì không ổn?
#283 3 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@decrypted-dev đang làm issue này rồi.

Từ ngày 16/10/2026.

  • #354 của @decrypted-dev — đang mở

Đánh giá

Issue này chưa được đánh giá.

Mô tả

area:pricing difficulty:easy epic:data-quality help wanted points:100 Stellar Wave
Background

src/routes/screener.ts sorts and filters on liquidity and nothing documents what it is. The same file already carries a scar from this: market_cap was once a copy of liquidity and the route now refuses it with an explanation. The remaining field has the same problem one step removed — a caller filtering liquidity >= 1000 has no idea what they asked for.

What to build
  • Define it, in the docs and in the spec: which reserves, valued in what, at which price, summed over which venues.
  • If the stored value does not match the definition, fix the value, not the definition.
  • Name the unit in the response.
Acceptance criteria
  • liquidity is defined in the spec with its unit and its sources
  • The stored value matches the definition, with a test
  • The response names the unit
  • Any change to the number is called out in the PR

Drips Wave · Complexity: Easy · 100 points

Repo: Divinity-Labs/Lens. Node + Fastify + Prisma + Postgres. Tests are vitest (npm test), not jest — jest against this repo fails every file and tells you nothing.

?network= is not optional on a new route. Every row carries a network and the indexes lead with it. Read req.network ?? activeNetwork, filter on it, echo it in the response, and declare the parameter in openapi.yaml. tests/openapi.test.ts boots the server and fails if a registered route is missing from the spec.

A number without provenance is not an answer. Every figure this batch adds must carry what it was computed from: which venues, how many samples, how fresh. The point of Lens is that its numbers can be checked.

Never elide a contract id or an issuer. Take addresses from the protocol's own documentation and write them whole. A sibling repo's issue wrote CD2W5LUR…XZ4L and five contributors invented plausible contract ids to fill the gap.

Say when you cannot answer. An asset with no liquidity, a pair with one venue, a window with a gap — each has an honest answer that is not zero and not a 500. Zero is a number people act on.

Required: Before submitting, join the contributor Telegram so your work can be tracked and counted toward the Stellar Wave: https://t.me/+fxHXq8f1SwlkZDBk

Ngôn ngữ chính
TypeScript
Star
4
Fork
71
Merge trung bình
1 ngày 15 giờ
Pull request đã merge (30 ngày)
28

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của Divinity-Labs/Lens

Tất cả issue của Divinity-Labs/Lens

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.