Skip to content

Conversation

@romeopopescu
Copy link
Contributor

Description

This PR fixes a typo in the Laravel SDK onboarding ("11.0 an up" -> "11.0 and up").

The fix was applied to:

  • static/app/gettingStartedDocs/php-laravel/onboarding.tsx
  • static/app/gettingStartedDocs/php-laravel/profiling.tsx

Note: I updated the source .tsx files but did not manually update the django.po translation files, assuming the build system will handle the synchronization.

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.

@romeopopescu romeopopescu requested a review from a team as a code owner November 24, 2025 16:23
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Nov 24, 2025
Copy link
Member

@shellmayr shellmayr left a comment

Choose a reason for hiding this comment

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

Thanks for fixing - as it's mentioned in the PR description, the tct() function handles translation automatically, so there's no need to update anything else.

@shellmayr shellmayr added the Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests label Nov 26, 2025
@shellmayr shellmayr merged commit 742439f into getsentry:master Nov 26, 2025
47 of 49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components Trigger: getsentry tests Once code is reviewed: apply label to PR to trigger getsentry tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants