Skip to content

[READY] Remove navbar links#605

Merged
Sharqiewicz merged 5 commits into
stagingfrom
remove-navbar-links
May 15, 2025
Merged

[READY] Remove navbar links#605
Sharqiewicz merged 5 commits into
stagingfrom
remove-navbar-links

Conversation

@prayagd
Copy link
Copy Markdown
Member

@prayagd prayagd commented Apr 30, 2025

  • Remove how it works
  • Remove community

- Remove how it works
- Remove community
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 30, 2025

Deploy Preview for pendulum-pay ready!

Name Link
🔨 Latest commit df30159
🔍 Latest deploy log https://app.netlify.com/sites/pendulum-pay/deploys/6824af784002fa0008f21117
😎 Deploy Preview https://deploy-preview-605--pendulum-pay.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 site configuration.

Copy link
Copy Markdown
Member

@ebma ebma left a comment

Choose a reason for hiding this comment

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

Was this change requested somewhere? I think if we touch the navbar, we also have to touch the last item. Keeping 'Sell Crypto' around doesn't make sense. Especially since it's called sell no matter if the user is in buy or sell mode.

@ebma ebma mentioned this pull request Apr 30, 2025
@ebma ebma marked this pull request as draft May 13, 2025 10:38
@Sharqiewicz Sharqiewicz marked this pull request as ready for review May 15, 2025 09:52
@Sharqiewicz Sharqiewicz changed the title Remove navbar links [READY] Remove navbar links May 15, 2025
Copy link
Copy Markdown
Member

@ebma ebma left a comment

Choose a reason for hiding this comment

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

LGTM

@Sharqiewicz Sharqiewicz merged commit 53361f8 into staging May 15, 2025
5 checks passed
@ebma ebma deleted the remove-navbar-links branch May 15, 2025 13:56
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.

3 participants