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

Update Challenge-simple-nft-example last Version Scaffold Stark #101

Merged
merged 52 commits into from
Aug 13, 2024

Conversation

Nadai2010
Copy link
Collaborator

Update last migration Scaffold Stark

Migrating the latest Scaffold Stark features PR232

Types of change

  • Feature
  • Bug
  • Enhancement

Comments (optional)

jrcarlos2000 and others added 30 commits July 11, 2024 16:42
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: 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]>
@Nadai2010 Nadai2010 marked this pull request as ready for review August 10, 2024 20:24
@Nadai2010 Nadai2010 requested a review from gianalarcon August 10, 2024 20:25
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"
Copy link
Collaborator

@gianalarcon gianalarcon Aug 13, 2024

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?: {
Copy link
Collaborator

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)
Copy link
Collaborator

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

@Nadai2010 Nadai2010 merged commit 4540ce1 into simple-nft-example Aug 13, 2024
1 check failed
@Nadai2010 Nadai2010 deleted the nadai/simple-nft-update branch August 13, 2024 18:50
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.