Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[iOS] - Properly handle URL eliding when blobs are given. (uplift to 1.71.x) #26096

Merged
merged 1 commit into from
Oct 21, 2024
Merged

[iOS] - Properly handle URL eliding when blobs are given. (uplift to 1.71.x) #26096

merged 1 commit into from
Oct 21, 2024

Conversation

brave-builds
Copy link
Collaborator

@brave-builds brave-builds commented Oct 18, 2024 •
edited by kjozwiak
Loading

Uplift of #26085
Resolves brave/brave-browser#41716

Pre-approval checklist:

  • You have tested your change on Nightly.
  • This contains text which needs to be translated.
    • There are more than 7 days before the release.
    • I've notified folks in #l10n on Slack that translations are needed.
  • The PR milestones match the branch they are landing to.

Pre-merge checklist:

  • You have checked CI and the builds, lint, and tests all pass or are not related to your PR.

Post-merge checklist:

  • The associated issue milestone is set to the smallest version that the changes is landed on.

@brave-builds brave-builds requested a review from a team as a code owner October 18, 2024 15:02
@brave-builds brave-builds added this to the 1.71.x - Release #2 milestone Oct 18, 2024
@brave-builds brave-builds self-assigned this Oct 18, 2024
@Brandon-T Brandon-T added CI/skip-android Do not run CI builds for Android CI/skip-macos-x64 Do not run CI builds for macOS x64 CI/skip-windows-x64 Do not run CI builds for Windows x64 unused-CI/skip-linux-x64 Do not run CI builds for Linux x64 CI/skip-macos-arm64 Do not run CI builds for macOS arm64 CI/skip-teamcity Do not run builds in TeamCity labels Oct 18, 2024
kjozwiak
kjozwiak approved these changes Oct 21, 2024
View reviewed changes
Copy link
Member

@kjozwiak kjozwiak left a comment

Choose a reason for hiding this comment

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

Uplift into 1.71.x approved 👍 QA has verified the PR on Nightly using 1.73.x as per #26085 (comment).

@kjozwiak kjozwiak merged commit 151fbd0 into 1.71.x Oct 21, 2024
16 checks passed
@kjozwiak kjozwiak deleted the pr26085_bugfix/URLElide-iOS_1.71.x branch October 21, 2024 01:18
@brave-builds
Copy link
Collaborator Author

Released in v1.71.116

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@kjozwiak kjozwiak kjozwiak approved these changes

Assignees

@brave-builds brave-builds

Labels
CI/skip-android Do not run CI builds for Android CI/skip-macos-arm64 Do not run CI builds for macOS arm64 CI/skip-macos-x64 Do not run CI builds for macOS x64 CI/skip-teamcity Do not run builds in TeamCity CI/skip-windows-x64 Do not run CI builds for Windows x64 unused-CI/skip-linux-x64 Do not run CI builds for Linux x64
Projects
None yet
Milestone
1.71.x - Release #2
Development

Successfully merging this pull request may close these issues.
3 participants
@brave-builds @kjozwiak @Brandon-T