Skip to content

Conversation

@kronosapiens
Copy link
Contributor

@kronosapiens kronosapiens commented Jul 23, 2025

Issue / feature description

This PR integrates the new ControllerConnector, making it easier to test future updates.

Changes

Checklist

  • Rebased to the last commit of the target branch (or merged)
  • Code self-reviewed
  • Code self-tested
  • Tests updated (if needed)
  • All tests are passing locally

@vercel
Copy link

vercel bot commented Jul 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
demo-dapp-starknet Ready Ready Preview Comment Sep 17, 2025 11:46am

@kronosapiens
Copy link
Contributor Author

Hi @bluecco, can you take a look at this PR?

@bluecco
Copy link
Contributor

bluecco commented Sep 10, 2025

hi @kronosapiens , we have @Cussone on it (along with starknetkit pr argentlabs/starknetkit#228)

@kronosapiens
Copy link
Contributor Author

Hey @Cussone, wanted to follow up on this

@Cussone
Copy link
Member

Cussone commented Sep 15, 2025

Hey @Cussone, wanted to follow up on this

Hey @kronosapiens, I am on it. However, I need to make some related changes first, since we're upgrading starknet.js to the next major version. It will be done in the next few days.

Sorry for making you wait this much, we were on offsite last week.

@Cussone Cussone self-requested a review September 17, 2025 11:40
# Conflicts:
#	package.json
#	pnpm-lock.yaml
@Cussone Cussone merged commit 1655c96 into argentlabs:develop Sep 17, 2025
2 checks passed
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.

Multi-connector compatibility issues when signing messages

3 participants