feat(epic): Create a walrus sites TS-SDK - WIP#618
Draft
Conversation
Publish a test site without using a site-builder CLI yet. Just use the walrus sites smart contract.
Move signing and gas handling into SiteBuilder.run and update tests to call run instead of having the flow sign/execute the transaction
Rename publish_site_flow to create_site_and_send_to_sender and update tests to import and call the new function. Add an empty publish_quilt stub in walrus utils.
Introduce CreateAndAddResourceOptions type and add a stub createAndAddResource method to WalrusSitesClient with TODO comments. Add publish implementation notes outlining the steps to create, attach, and add resources; keep other site methods unimplemented.
Running the publish test returns a timeout error. Maybe we should consider using an upload relay?
Added a few more stuff to the client.publish but it fails with a RangeError (not a number) from inside the createAndAddResource. I should create a unit tests to debug this.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…ansactionExecutor, for simple Signer.signAndExecuteTransaction, because it wasn't working with codegen mvr types
|
Tzal3x
requested review from
nikos-terzo
and removed request for
nikos-terzo
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Important notice
This branch is considered an epic. Please merge all your TS-SDK related PRs to this branch until we finally release it.
Summary
prettierrcfor linting, integrate sonarrqube.