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.

Sign up for GitHub

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

Jump to bottom

Contact Us improvements #5872

Merged
merged 3 commits into from
Mar 27, 2024
Merged

Contact Us improvements #5872

merged 3 commits into from
Mar 27, 2024

Conversation

dennisreimann
Copy link
Member

ndeet
ndeet approved these changes Mar 26, 2024
View reviewed changes
Copy link
Contributor

@ndeet ndeet left a comment

Choose a reason for hiding this comment

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

Contact us looks nice and the obfuscation works beatifully when javascript disabled. Thanks a lot 👍

@dstrukt dstrukt self-requested a review March 27, 2024 00:05
dstrukt
dstrukt approved these changes Mar 27, 2024
View reviewed changes
Copy link
Member

@dstrukt dstrukt left a comment

Choose a reason for hiding this comment

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

tACK - LGTM, thinking we'll want to restructure the Account menu a bit in the future, as it's grown quite a bit, but works well right now!

@NicolasDorier NicolasDorier merged commit 0f3f8b6 into btcpayserver:master Mar 27, 2024
4 checks passed
@dennisreimann dennisreimann deleted the contact-us branch March 27, 2024 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dstrukt dstrukt dstrukt approved these changes

@ndeet ndeet ndeet approved these changes

Assignees
No one assigned
Labels
Urgent
Projects
None yet
Milestone
No milestone
5 participants
@dennisreimann @ndeet @dstrukt @NicolasDorier @pavlenex