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

docs: /paras{/leases/current, /auctions/current} #507

Merged
merged 8 commits into from
Apr 9, 2021

Conversation

emostov
Copy link
Contributor

@emostov emostov commented Apr 7, 2021

adds docs for

  • /paras
  • /paras/leases/current
  • /paras/auctions/current
  • /paras/crowdloans
  • /paras/{paraId}/crowdloan-info
  • /paras/{paraId}/lease-info

changes

  • Auction phases are now modeled as AuctionPhase::Starting instead of (AuctionPhase::PreEnding) and AuctionPhase::Ending.

future prs

  • tests

@emostov emostov requested a review from TarikGul April 7, 2021 01:48
@emostov
Copy link
Contributor Author

emostov commented Apr 7, 2021

@joepetrowski this might be a good time to take a peek at how the api is developing. You can see the response interfaces at src/types/responses/Paras.ts

@emostov emostov marked this pull request as draft April 8, 2021 03:28
@emostov emostov marked this pull request as ready for review April 8, 2021 06:20
[Experimental subject to breaking change] Get the status of the current
auction.
description: |
Returns an overview of the current of the current auction. There is only
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Returns an overview of the current of the current auction. There is only
Returns an overview of the current auction. There is only

format: unsignedInteger or $hex
- name: fundInfo
in: query
description: Wehther or not to include the `fundInfo` of each crowdlaon
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
description: Wehther or not to include the `fundInfo` of each crowdlaon
description: Optional parameter to include the `fundInfo` of each crowdlaon

parameters:
- name: paraId
in: path
description: paraId to query crowdloan information of.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sounds a little weird. It might just be the of at the end.

- paras
summary: |
[Experimental subject to breaking change] Get current and future lease
info + lifecycle stage for a given `paraId.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
info + lifecycle stage for a given `paraId.
info as well as the lifecycle stage for a given `paraId.

Optional suggestion. Not sure if having symbols is conventional for docs.

parameters:
- name: paraId
in: path
description: paraId to query crowdloan information of.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same thing as above with the of.

type: string
format: unsignedInteger
description: |
A currently win bid and the set of lease periods the bid is for. The
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
A currently win bid and the set of lease periods the bid is for. The
A currently won bid and the set of lease periods the bid is for. The

type: string
format: unisgnedInteger or $null
description: |
Fist block of the auction ending phase. `null` if no ongoing auction.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Fist block of the auction ending phase. `null` if no ongoing auction.
Fist block of the auction ending phase. Returns `null` if there is no ongoing auction.

type: string
format: unisgnedInteger or $null
description: |
Last block of the auction ending phase. `null` if no ongoing auction.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Last block of the auction ending phase. `null` if no ongoing auction.
Last block of the auction ending phase. Returns `null` if there is no ongoing auction.

- opening
- ending
description: |
Wether the auction is in the `opening` or `ending` phase. The
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Wether the auction is in the `opening` or `ending` phase. The
Whether the auction is in the `opening` or `ending` phase. The

description: |
Wether the auction is in the `opening` or `ending` phase. The
`ending` phase is where the eventual winners are retroactively
picked from. `null` if no ongoing auction.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
picked from. `null` if no ongoing auction.
picked from. Returns `null` if there is no ongoing auction.

type: string
format: unsignedInteger
description: |
Lease period indexes that may be bid on in this auction. `null` if no
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Lease period indexes that may be bid on in this auction. `null` if no
Lease period indexes that may be bid on in this auction. Returns `null` if there is no

fundInfo:
$ref: '#/components/schemas/FundInfo'
description: |
List of paras that have crowdloans. Note `fundInfo` is only included when query
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
List of paras that have crowdloans. Note `fundInfo` is only included when query
List of paras that have crowdloans. Note `fundInfo` is only included when the query

Copy link
Member

@TarikGul TarikGul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just some grammar, and wording suggestions.

@emostov emostov merged commit 722d756 into feat-experiment-plo-endpoints Apr 9, 2021
@emostov emostov deleted the zeke-plo-docs-1 branch April 9, 2021 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants