WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@vpremamozilla
Copy link
Collaborator

@vpremamozilla vpremamozilla commented Dec 1, 2025

MPP-4497

How to test

Screenshot (if applicable)

Screenshot 2025-11-30 at 5 58 14 PM

Checklist (Definition of Done)

  • Product Owner accepted the User Story (demo of functionality completed) or waived the privilege.
  • Customer Experience team has seen or waived a demo of functionality.
  • All acceptance criteria are met.
  • Jira ticket has been updated (if needed) to match changes made during the development process.
  • I've added or updated relevant docs in the docs/ directory
  • Jira ticket has been updated (if needed) with suggestions for QA when this PR is deployed to stage.
  • All UI revisions follow the coding standards, and use Protocol / Nebula colors where applicable (see /frontend/src/styles/colors.scss).
  • Commits in this PR are minimal and have descriptive commit messages.
  • l10n changes have been submitted to the l10n repository, if any.

@vpremamozilla vpremamozilla force-pushed the MPP-4497-update-relay-vpn-onboarding-page branch 2 times, most recently from c642c87 to 937fd91 Compare December 1, 2025 18:28
Copy link
Collaborator

@joeherm joeherm left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Member

@jwhitlock jwhitlock left a comment

Choose a reason for hiding this comment

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

Thanks @vpremamozilla! One question, maybe vpn-relay-go-relay-body-3 should bevpn-relay-go-relay-body-2

phone-onboarding-step1-button-cta-2 = Sign up
phone-onboarding-step1-learn-more = Learn more
vpn-relay-go-relay-body-2 = Keep your identity private
Copy link
Member

Choose a reason for hiding this comment

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

Is vpn-relay-go-relay-body-2 used? I can't find it in the code.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good callout, i will remove

@vpremamozilla vpremamozilla force-pushed the MPP-4497-update-relay-vpn-onboarding-page branch from 937fd91 to acd3805 Compare December 1, 2025 18:56
@vpremamozilla
Copy link
Collaborator Author

Thanks @vpremamozilla! One question, maybe vpn-relay-go-relay-body-3 should bevpn-relay-go-relay-body-2

unfortunately, the l10n PR was already merged so had to use 3

@vpremamozilla vpremamozilla added this pull request to the merge queue Dec 1, 2025
Merged via the queue into main with commit 95a1067 Dec 1, 2025
31 checks passed
@vpremamozilla vpremamozilla deleted the MPP-4497-update-relay-vpn-onboarding-page branch December 1, 2025 20:34
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