Skip to content

User Profile Modal Navigation & wallet management#152

Merged
DioChuks merged 1 commit intoStarkBid:mainfrom
0xDeon:fix-145-navigation
Aug 29, 2025
Merged

User Profile Modal Navigation & wallet management#152
DioChuks merged 1 commit intoStarkBid:mainfrom
0xDeon:fix-145-navigation

Conversation

@0xDeon
Copy link
Contributor

@0xDeon 0xDeon commented Aug 29, 2025

Description

Resolves the issue of no routes for the user's profile modal, when the user connects with their starknet wallet.

Analysis

this involved thinking about the current structure & layout of the user wallet signature process in respect to the auth module, the profile modal navigation items.

concerns

  • some of the routes like offers/bids require their own overview page
  • current user profile page needs implementation
  • the verification badge system

screenshots

image

Closes #145

@netlify
Copy link

netlify bot commented Aug 29, 2025

Deploy Preview for starkbid ready!

Name Link
🔨 Latest commit 6afb6c3
🔍 Latest deploy log https://app.netlify.com/projects/starkbid/deploys/68b2164418cd670007a9296e
😎 Deploy Preview https://deploy-preview-152--starkbid.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Aug 29, 2025

Deploy Preview for starkbid-stable ready!

Name Link
🔨 Latest commit 6afb6c3
🔍 Latest deploy log https://app.netlify.com/projects/starkbid-stable/deploys/68b21644441d0900084c2823
😎 Deploy Preview https://deploy-preview-152--starkbid-stable.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@DioChuks DioChuks self-requested a review August 29, 2025 21:31
Copy link
Contributor

@DioChuks DioChuks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great Insights, your concerns are valid. I would check 'em out and see how they cling to the current system.

@DioChuks DioChuks merged commit 4bd5a1d into StarkBid:main Aug 29, 2025
8 of 9 checks passed
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.

Ensure Correct Routing for All Navigation Links in Profile Modal

2 participants