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

Integration with Scaffold-stark: Updated develop branch to use latest version of scaffold-stark #112

Merged
merged 69 commits into from
Dec 30, 2024

Conversation

michojekunle
Copy link
Contributor

@michojekunle michojekunle commented Dec 20, 2024

Integration with Scaffold-stark: Updated develop branch to use latest version of scaffold-stark

Fixes #93

Types of change

  • Feature
  • Bug
  • Enhancement

Comments (optional)

@michojekunle
Copy link
Contributor Author

Hi @gianalarcon I've made the requested changes regarding this issue #93 linked to this PR #111

Please review

@gianalarcon
Copy link
Collaborator

yarn compile & yarn test not working
image

@michojekunle
Copy link
Contributor Author

referral codes: OPBDFRSU J3KE0YMR

@michojekunle

Thank you ser

@michojekunle
Copy link
Contributor Author

Hi @gianalarcon It just seems like I'm stuck here i think it's the api production uri I'm missing from the env

image

@gianalarcon
Copy link
Collaborator

DM on discord pls https://discord.gg/bWuQFHHA

@michojekunle
Copy link
Contributor Author

michojekunle commented Dec 26, 2024

DM on discord pls https://discord.gg/bWuQFHHA

Alright, done

@Buuh2511
Copy link
Collaborator

  • Put text as a text-center here pls
    image

  • It will take a short while to see the balance, it is going from 0 to ...STRK, during this time do loading as a Spiner

Then, everything is working for now, I will check more round in morning tomorrow

@Buuh2511
Copy link
Collaborator

Buuh2511 commented Dec 26, 2024

  • Put text as a text-center here pls
    image
  • It will take a short while to see the balance, it is going from 0 to ...STRK, during this time do loading as a Spiner

Then, everything is working for now, I will check more round in morning tomorrow

Also need to implement for Select Token Modal

@michojekunle
Copy link
Contributor Author

Hi @Buuh2511

I've now added in both pages (deposit and withdrawal) and the select token modal the loading spinner while fetching token balances

Please review

@Buuh2511
Copy link
Collaborator

Spiner just keep loading, cant get balance in somtime
Screenshot 2024-12-27 at 10 03 36 AM

Need to fix alignment
Screenshot 2024-12-27 at 10 04 01 AM

When try to connect wallet without extensions going got bug
Screenshot 2024-12-27 at 10 06 47 AM

Got an error when try to connect wallet but still havent connected yet at the first time launch app
Screenshot 2024-12-27 at 10 15 51 AM

@michojekunle
Copy link
Contributor Author

Spiner just keep loading, cant get balance in somtime Screenshot 2024-12-27 at 10 03 36 AM

Need to fix alignment Screenshot 2024-12-27 at 10 04 01 AM

When try to connect wallet without extensions going got bug Screenshot 2024-12-27 at 10 06 47 AM

Got an error when try to connect wallet but still havent connected yet at the first time launch app Screenshot 2024-12-27 at 10 15 51 AM

For the loading spinner it's only when I'm not connected that it keeps showing spinning or whether there's an error fetching the balance it shows the spinning

here first cause my wallet is not connected yet
PS: I refreshed the page

image

And then here where my wallet is now reflected as connected after complete page load and I'm not logged in so my marquis balance keeps spinning cause of the error fetching my balance
image

  • Fixed the last two bugs

  • Fixed this alignment also
    image

@michojekunle
Copy link
Contributor Author

Hi, Gm @Buuh2511

Please review

@Buuh2511
Copy link
Collaborator

  • got this error when try to connect walllet without extension
    image

  • "PS: I refreshed the page" : so, we need to keep connection after reload page, better ux

  • Cant close modal when click on left and right of screen, only close when click on top and bottom of screen
    image

@michojekunle
Copy link
Contributor Author

michojekunle commented Dec 28, 2024

  • got this error when try to connect walllet without extension
    image
  • "PS: I refreshed the page" : so, we need to keep connection after reload page, better ux
  • Cant close modal when click on left and right of screen, only close when click on top and bottom of screen
    image

Alright Gm @Buuh2511

  • So, the first error means that the wallet is not installed in the current user browser that is the connector (argent or braavos) is not found

  • yes the connection is kept, it just takes a little time to restore the connection locally nextjs dev server loading speed

  • Alright will look into this last error now for the modal

@Buuh2511
Copy link
Collaborator

Buuh2511 commented Dec 28, 2024

  • got this error when try to connect walllet without extension
    image
  • "PS: I refreshed the page" : so, we need to keep connection after reload page, better ux
  • Cant close modal when click on left and right of screen, only close when click on top and bottom of screen
    image

Alright Gm @Buuh2511

  • So, the first error means that the wallet is not installed in the current user browser that is the connector (argent or braavos) is not found
  • yes the connection is kept, it just takes a little time to restore the connection locally nextjs dev server loading speed
  • Alright will look into this last error now for the modal

"PS: I refreshed the page" : so, we need to keep connection after reload page, better ux
i mean when switch between deposit and withdraw page, connection should be kepth

@michojekunle
Copy link
Contributor Author

  • got this error when try to connect walllet without extension
    image
  • "PS: I refreshed the page" : so, we need to keep connection after reload page, better ux
  • Cant close modal when click on left and right of screen, only close when click on top and bottom of screen
    image

Alright Gm @Buuh2511

  • So, the first error means that the wallet is not installed in the current user browser that is the connector (argent or braavos) is not found
  • yes the connection is kept, it just takes a little time to restore the connection locally nextjs dev server loading speed
  • Alright will look into this last error now for the modal

"PS: I refreshed the page" : so, we need to keep connection after reload page, better ux
i mean when switch between deposit and withdraw page, connection should be kepth

Alright will look into this too

@michojekunle
Copy link
Contributor Author

michojekunle commented Dec 28, 2024

Hi @Buuh2511

  • I think the wallet connection kept works well with braavos wallet, this only comes up with argent wallet in this case any time the dapp wants to reestablish a connection to argent wallet it prompts users to reenter their passwords to unlock their argent wallet and I read about that in this issue Feature Request: Expose Wallet Lock State & Disconnect Event starknet-io/get-starknet#192, which a user will most likely reject there and when the wallet is locked the dapp cannot reestablish connection with the wallet

@michojekunle
Copy link
Contributor Author

  • Also I have now fixed the invitation modal to close on click on the sides @Buuh2511

Please Review

@Buuh2511 Buuh2511 self-requested a review December 30, 2024 03:18
@Buuh2511
Copy link
Collaborator

Many thanks for your contribution ! @michojekunle

@gianalarcon gianalarcon merged commit 8331f86 into The-Marquis-Gaming:test-develop Dec 30, 2024
2 checks passed
@michojekunle
Copy link
Contributor Author

Many thanks for your contribution ! @michojekunle

Thank you so much @gianalarcon @Buuh2511 for the opportunity to contribute, It's really nice contributing to this project and working with you guys 🙌🏾❤️, Really learnt a lot working on this 💯

Looking forward to contributing more 🚀🚀

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.

4 participants