Skip to content

Conversation

@0xfirefist
Copy link
Contributor

Completed

  • init package
  • move all the data structures to the package and consume them in the contract
  • add MockPyth to facilitate testing for pyth contract consumers

Pending (to cover in other PRs)

  • any helper function if required
  • generate schema
  • setup release pipeline
  • update contract to consume pyth-sdk-cw instead of pyth-cosmwasm.

@vercel
Copy link

vercel bot commented Feb 13, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Ignored Deployments
Name Status Preview Updated
example-oracle-amm ⬜️ Ignored (Inspect) Feb 13, 2023 at 9:30AM (UTC)
xc-admin-frontend ⬜️ Ignored (Inspect) Feb 13, 2023 at 9:30AM (UTC)

}

/// TODO: Update this with example contracts -> Handler for processing query messages. See the tests in `contract.rs` for how to use this
/// handler within your tests.
Copy link
Contributor

Choose a reason for hiding this comment

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

i note that contract.rs isn't included here at the moment. if you're going to add that in a future PR, great. If not, we should document how to use this somewhere.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, the example contract here - https://github.com/pyth-network/pyth-crosschain/blob/main/target_chains/cosmwasm/examples/cw-contract/src/contract.rs

Will update the example in future PRs. This Todo will be resolved then.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Merging this rn

@0xfirefist 0xfirefist merged commit 66a46dd into main Feb 13, 2023
@0xfirefist 0xfirefist deleted the add-module-testing branch February 13, 2023 16:36
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