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

Fix usertext comment to ensure it matches localizable string #2546

Merged
merged 2 commits into from
Apr 4, 2024

Conversation

aataraxiaa
Copy link
Contributor

@aataraxiaa aataraxiaa commented Apr 4, 2024

Copy link
Contributor

@diegoreymendez diegoreymendez left a comment

Choose a reason for hiding this comment

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

Consider disabling the failing tests and adding them to the list of flaky tests.

See https://app.asana.com/0/1200194497630846/1203329782697866/f.

On my end this is good.

@aataraxiaa aataraxiaa merged commit 35c725d into main Apr 4, 2024
17 checks passed
@aataraxiaa aataraxiaa deleted the pete/fix-usertext-comment branch April 4, 2024 13:27
samsymons pushed a commit that referenced this pull request Apr 5, 2024
Updates UserText string with comment to match localizable file comment.
samsymons added a commit that referenced this pull request Apr 8, 2024
* main: (338 commits)
  Fixes the VPN restarting logic on update (#2545)
  fix download save panel disappearing on navigation (#2549)
  Add CI support for handling installation attribution (#2502)
  Bump version to 1.82.0 (152)
  Pixel changed in subscription (#2541)
  add LSHandlerRank for .duckload document type (#2537)
  Fix usertext comment to ensure it matches localizable string (#2546)
  Update Neighbor Report (#2542)
  DBP: Compare by url and not name (#2544)
  DBP: Compare by url and not name (#2544)
  Adds a series of UI tests for Bookmarks Bar visibility
  Improve Handling of noData Import Errors (#2494)
  Bump version to 1.82.0 (151)
  Use the default action button style for VPN onboarding (#2529)
  Redirect from purchase page to macOS native purchase flow (#2538)
  Closing empty tabs after download (#2510)
  Add Web UI loading state pixels (#2531)
  fix localization warnings (#2288)
  Select URL in address bar when editing and use cmd+L (#2536)
  Add attemptId to Captcha /result endpoint (#2530)
  ...
samsymons added a commit that referenced this pull request Apr 8, 2024
* main: (22 commits)
  Removes last instance of NETWORK_PROTECTION flag (#2573)
  Bye bye NETWORK_PROTECTION (#2509)
  QWD: Enable Hide/Show for Autofill Credit Card Number and CVV (#2539)
  Removed the VPN waitlist beta pixels (#2555)
  VPN: Cleanup authorize call (#2565)
  Revert "VPN: Cleanup authorize call (#2553)"
  VPN: Cleanup authorize call (#2553)
  Improves underlying error pixel information (#2543)
  Fixes the VPN restarting logic on update (#2545)
  fix download save panel disappearing on navigation (#2549)
  Add CI support for handling installation attribution (#2502)
  Fix usertext comment to ensure it matches localizable string (#2546)
  Update Neighbor Report (#2542)
  DBP: Compare by url and not name (#2544)
  Adds a series of UI tests for Bookmarks Bar visibility
  Improve Handling of noData Import Errors (#2494)
  Use the default action button style for VPN onboarding (#2529)
  Closing empty tabs after download (#2510)
  Add Web UI loading state pixels (#2531)
  fix localization warnings (#2288)
  ...
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.

2 participants