-
Notifications
You must be signed in to change notification settings - Fork 18
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
Update Challenge-simple-nft-example last Version Scaffold Stark #101
Conversation
Co-authored-by: 0xquantum3labs <[email protected]> Co-authored-by: jrcarlos2000 <[email protected]>
Co-authored-by: 0xquantum3labs <[email protected]> Co-authored-by: jrcarlos2000 <[email protected]> Co-authored-by: gianmalarcon <[email protected]>
Co-authored-by: Eduardo <[email protected]>
…inor] Co-authored-by: Eduardo <[email protected]>
Co-authored-by: Eduardo <[email protected]>
Co-authored-by: jrcarlos2000 <[email protected]> Co-authored-by: gianalarcon <[email protected]> Co-authored-by: Eduardo <[email protected]> Co-authored-by: Eduardo <[email protected]> Co-authored-by: Gabriela del Pilar Rojas <[email protected]> Co-authored-by: Richard <[email protected]> Co-authored-by: Carlos Ramos <[email protected]>
Co-authored-by: CarlosR <[email protected]>
Co-authored-by: CarlosR <[email protected]>
Co-authored-by: CarlosR <[email protected]>
Co-authored-by: Carlos Ramos <[email protected]>
Co-authored-by: Carlos Ramos <[email protected]> Co-authored-by: Gabriela del Pilar Rojas <[email protected]> Co-authored-by: 0xquantum3labs <[email protected]> Co-authored-by: jrcarlos2000 <[email protected]> Co-authored-by: Daniel Calderón Díaz <[email protected]> Co-authored-by: gianmalarcon <[email protected]> Co-authored-by: Ugonna Paul Dike <[email protected]> Co-authored-by: Eduardo <[email protected]> Co-authored-by: Eduardo <[email protected]> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: oceanlvr <[email protected]> Co-authored-by: Quantum3Labs <[email protected]> Co-authored-by: gianalarcon <[email protected]> Co-authored-by: Gabriela del Pilar Rojas <[email protected]> Co-authored-by: Richard <[email protected]> Co-authored-by: CarlosR <[email protected]> Co-authored-by: Abdulhakeem Abdulazeez Ayodeji <[email protected]> Co-authored-by: Ikem <[email protected]> Co-authored-by: Wolf <[email protected]> Co-authored-by: Joseph Chimebuka <[email protected]>
@@ -151,7 +151,7 @@ export const Header = () => { | |||
alt="SE2 logo" | |||
className="cursor-pointer" | |||
fill | |||
src="/challenge-icon-starknet.svg" |
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.
this two .svg are the same for me. We can delete one of them. Lets check and fix it from the scaffold-stark repo. We can fix this from the challenge-base
branch
@@ -34,5 +34,9 @@ export const useScaffoldReadContract = < | |||
...(readConfig as any), | |||
}) as Omit<ReturnType<typeof useContractRead>, "data"> & { | |||
data: AbiFunctionOutputs<ContractAbi, TFunctionName> | undefined; | |||
refetch: (options?: { |
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.
we removed this before, and it is not part of scaffold-stark now
@@ -123,7 +123,7 @@ Prepare your environment variables. | |||
|
|||
> 🦊 Since we have deployed to a public testnet, you will now need to connect using a wallet you own(Argent X or Braavos). | |||
|
|||
![connect-wallet](https://raw.githubusercontent.com/Quantum3-Labs/speedrunstark/gabi/simple-nft-example/packages/nextjs/public/ch0-wallet.png) | |||
![connect-wallet](https://raw.githubusercontent.com/Quantum3-Labs/speedrunstark/simple-nft-example/packages/nextjs/public/ch0-wallet.png) |
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.
remove reference to https link, lets call relative/internal path
Update last migration Scaffold Stark
Migrating the latest Scaffold Stark features PR232
Types of change
Comments (optional)