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

fix: token check bug #286

Merged
merged 2 commits into from
Dec 3, 2024

Conversation

ugur-eren
Copy link
Collaborator

@ugur-eren ugur-eren commented Dec 3, 2024

Resolves a bug introduced with the PR #282 which upgrades the starknetjs version but ignores @starknet-react/core and @starknet-react/chains packages.
I had to make changes to a few files to be able upgrade it, since there are API changes with the newer versions.
I also fixed minor warnings in the tg-bot package.
I also created 2 new CI jobs for core & hooks packages.

Package version changes

  • starknet peer dependency upgraded from >=5.0.0 to >=6.8.0
  • @starknet-react/core dependency upgraded from 2.2.4 to 3.6.2
  • @starknet-react/core peer dependency upgraded from >=2.0.0 to >=3.0.0
  • @starknet-react/chains dependency upgraded from 0.1.0 to 3.1.0
  • @tanstack/react-query dependency upgraded from 5.0.1 to 5.25.0
  • @starknet-io/types-js dependency installed with version ^0.7.10

Copy link

vercel bot commented Dec 3, 2024

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

Name Status Preview Comments Updated (UTC)
unruggable-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 3, 2024 1:13am

Copy link

vercel bot commented Dec 3, 2024

@ugur-eren is attempting to deploy a commit to the keep-starknet-strange Team on Vercel.

A member of the Team first needs to authorize it.

@ugur-eren ugur-eren merged commit 7e83489 into keep-starknet-strange:main Dec 3, 2024
6 of 7 checks passed
@ugur-eren ugur-eren deleted the fix/token-check-bug branch December 4, 2024 07:37
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.

2 participants