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

Change redactAllText and redactAllImages to maskAllText and maskAllImages #12537

Merged
merged 1 commit into from
Feb 4, 2025

Conversation

ademirev
Copy link
Contributor

@ademirev ademirev commented Feb 2, 2025

This is based on the change in getsentry/sentry-java#3741

DESCRIBE YOUR PR

  • redactAllText and redactAllImages are no longer valid in Xcode, and have been replaced with maskAllText and maskAllImages

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.

  • Urgent deadline (GA date, etc.):
  • Other deadline:
  • [ X] None: Not urgent, can wait up to 1 week+

LEGAL BOILERPLATE

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

…ages

This is based on the change in getsentry/sentry-java#3741

redactAllText and redactAllImages are no longer valid in XCode, and have been replaced with maskAllText and maskAllImages
Copy link

vercel bot commented Feb 2, 2025

@ademirev is attempting to deploy a commit to the Sentry Team on Vercel.

A member of the Team first needs to authorize it.

@coolguyzone
Copy link
Contributor

@philipphofmann mind taking a look?

Copy link
Member

@philipphofmann philipphofmann left a comment

Choose a reason for hiding this comment

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

Good catch thanks @ademirev 💯

@philipphofmann
Copy link
Member

@coolguyzone, I can't authorize the required CI checks so we can merge the PR. Can you do that?

Copy link

vercel bot commented Feb 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 4, 2025 7:11pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
changelog ⬜️ Ignored (Inspect) Visit Preview Feb 4, 2025 7:11pm
develop-docs ⬜️ Ignored (Inspect) Visit Preview Feb 4, 2025 7:11pm

@coolguyzone
Copy link
Contributor

Thanks @philipphofmann , I will run the builds and merge 🏄‍♂️

@coolguyzone coolguyzone enabled auto-merge (squash) February 4, 2025 18:58
@coolguyzone coolguyzone merged commit 77ef7e5 into getsentry:master Feb 4, 2025
11 checks passed
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