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

Add API support for queries for protocol params. #2263

Merged
merged 5 commits into from
Jan 11, 2021

Conversation

dcoutts
Copy link
Contributor

@dcoutts dcoutts commented Jan 7, 2021

Several kinds of params:

  1. current values of updateable protocol params
  2. any pending proposed param updates
  3. the non-updateable params fixed in the (Shelley) genesis file

@dcoutts
Copy link
Contributor Author

dcoutts commented Jan 7, 2021

The first two patches are from PR 2238. This can be rebased once they're merged.

Before we add more things into that module.
To go along with the existing ProtocolParametersUpdate type. This new
one will be used for the query that returns the current param values.
The node keeps the values of the params that are fixed in the genesis
file.

-- Please enter the commit message for your changes. Lines starting
Several kinds of params:
1. current values of updateable protocol params
2. any pending proposed param updates
3. the non-updateable params fixed in the (Shelley) genesis file
@dcoutts dcoutts force-pushed the dcoutts/api-ipc-query-yet-more branch from aa83a0d to b5abac0 Compare January 7, 2021 03:16
-- Genesis paramaters
--

data GenesisParameters =
Copy link
Contributor

Choose a reason for hiding this comment

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

💯

-- :: Set StakeAddress
-- -> QueryInShelleyBasedEra (Map StakeAddress Lovelace,
-- Map StakeAddress PoolId)
QueryStakeAddresses
Copy link
Contributor

@Jimbo4350 Jimbo4350 Jan 8, 2021

Choose a reason for hiding this comment

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

@dcoutts We are duplicating work: 5355043

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops. You can take over this PR.

@Jimbo4350
Copy link
Contributor

bors merge

@iohk-bors
Copy link
Contributor

iohk-bors bot commented Jan 11, 2021

@iohk-bors iohk-bors bot merged commit e3632a3 into master Jan 11, 2021
@iohk-bors iohk-bors bot deleted the dcoutts/api-ipc-query-yet-more branch January 11, 2021 13:16
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.

3 participants