-
Notifications
You must be signed in to change notification settings - Fork 3
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(deps) Update starknet-reacts next to 3.0.0-beta.3 #37
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Decouple components using different lib versions.
Decouple components using different lib versions.
|
||
const buttonsDisabled = ["approve", "pending"].includes(transactionStatus) | ||
|
||
const { contract } = useContract({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
using new api from newest version of starknet-react
"starknet-react-chains-next": "npm:@starknet-react/[email protected].3", | ||
"starknet-react-core-next": "npm:@starknet-react/[email protected].3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if we should set it to this version or will they release the final one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess there shouldn't be any differences, just curious
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
idk to be honest, but it is a good question
|
||
export const MintWithStarknetReact = () => { | ||
const { account } = useAccount() | ||
const [mintAmount, setMintAmount] = useState("10") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be prepopulated with 10
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a copy-paste from previous component, so didn't want to change it. I assume it's just some random default
value={mintAmount} | ||
onChange={(e) => setMintAmount(e.target.value)} | ||
/> | ||
{/* TODO: When will we allow below? Need to ask Ale */} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why would we disable it at all, after all it's just an example dapp?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I asked Ale and blockchian team. He basically said it's always been there. Going yolo and enabling the submit 😂
<Input | ||
type="submit" | ||
disabled={true || buttonsDisabled} | ||
value="Not possible with ETH!" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
value="Not possible with ETH!"
what does this mean?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I asked Ale and blockchian team. He basically said it's always been there. Going yolo and enabling the submit 😂
export const TransferWithStarknetReact = () => { | ||
const { account } = useAccount() | ||
const [transferTo, setTransferTo] = useState("") | ||
const [transferAmount, setTransferAmount] = useState("1") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
value 1
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above
No description provided.