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.

Sign up for GitHub

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

Jump to bottom

[Clients] embed ParamsQuerier into SupplierQueryClient #1001

Draft
wants to merge 1 commit into
base: issues/543/params-querier/session
Choose a base branch
from
Draft

[Clients] embed ParamsQuerier into SupplierQueryClient #1001

wants to merge 1 commit into from

Conversation

bryanchriswhite
Copy link
Contributor

Summary

Embed the ParamsQuerier interface into SupplierQueryClient and update the implementations.

Issue

Type of change

Select one or more from the following:

Testing

  • Documentation: make docusaurus_start; only needed if you make doc changes
  • Unit Tests: make go_develop_and_test
  • LocalNet E2E Tests: make test_e2e
  • DevNet E2E Tests: Add the devnet-test-e2e label to the PR.

Sanity Checklist

  • I have tested my changes using the available tooling
  • I have commented my code
  • I have performed a self-review of my own code; both comments & source code
  • I create and reference any new tickets, if applicable
  • I have left TODOs throughout the codebase, if applicable

@bryanchriswhite bryanchriswhite added supplier Changes related to the Supplier actor off-chain Off-chain business logic labels Dec 11, 2024
@bryanchriswhite bryanchriswhite self-assigned this Dec 11, 2024
@bryanchriswhite bryanchriswhite force-pushed the issues/543/params-querier/shared branch from a19f686 to 1e3af86 Compare December 11, 2024 16:28
@bryanchriswhite bryanchriswhite force-pushed the issues/543/params-querier/supplier branch from e91c0c7 to 526615e Compare December 11, 2024 17:05
@bryanchriswhite bryanchriswhite force-pushed the issues/543/params-querier/supplier branch from 526615e to a2ff713 Compare December 11, 2024 17:11
@bryanchriswhite bryanchriswhite linked an issue Dec 11, 2024 that may be closed by this pull request
[Off-Chain] ModuleParamsClient & Historical Params #543
Open
4 tasks
@bryanchriswhite bryanchriswhite force-pushed the issues/543/params-querier/shared branch 5 times, most recently from d52af81 to d630484 Compare December 13, 2024 14:31
@bryanchriswhite bryanchriswhite changed the base branch from issues/543/params-querier/shared to issues/543/params-querier/session December 13, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@bryanchriswhite bryanchriswhite

Labels
off-chain Off-chain business logic supplier Changes related to the Supplier actor
Projects
Shannon
Status: 🏗 In progress
Milestone
Shannon Beta TestNet Support
Development

Successfully merging this pull request may close these issues.

[Off-Chain] ModuleParamsClient & Historical Params
1 participant
@bryanchriswhite