Skip to content

Fix 'occured' -> 'occurred' typos in user-visible strings across multiple plugins#263916

Closed
SAY-5 wants to merge 8 commits intoelastic:mainfrom
SAY-5:fix-occured-typos-sources
Closed

Fix 'occured' -> 'occurred' typos in user-visible strings across multiple plugins#263916
SAY-5 wants to merge 8 commits intoelastic:mainfrom
SAY-5:fix-occured-typos-sources

Conversation

@SAY-5
Copy link
Copy Markdown

@SAY-5 SAY-5 commented Apr 16, 2026

Summary

Fix occuredoccurred typos across 7 Kibana source files. Most are user-visible error strings surfaced in UI components or log output.

Files

Plugin File Kind
canvas function_form.tsx:51 UI error label
apm error_state_prompt.tsx:18 defaultMessage (user-visible in UI)
security record_violations.ts:101,105 JSDoc
search_synonyms error_prompt.tsx:21 defaultMessage (user-visible)
ingest_pipelines mapper.ts:67 Error response body
observability_ai_assistant insight_error.tsx:22 defaultMessage (user-visible)
task_manager task_store.ts:537,690 Log string

Testing

String-literal-only change in all cases. No logic, no API, no translation keys — only the English source strings in defaultMessage/error text. i18n extraction pipelines re-generate from source, so this just produces cleaner English output.

SAY-5 added 7 commits April 16, 2026 14:28
Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
Signed-off-by: SAY-5 <SAY-5@users.noreply.github.com>
@SAY-5 SAY-5 requested review from a team as code owners April 16, 2026 21:29
@SAY-5 SAY-5 requested a review from kc13greiner April 16, 2026 21:29
@botelastic botelastic Bot added ci:project-deploy-observability Create an Observability project Team:obs-ai Observability AI team Team:obs-presentation Focus: APM UI, Infra UI, Hosts UI, Universal Profiling, Obs Overview and left Navigation labels Apr 16, 2026
@cla-checker-service
Copy link
Copy Markdown

cla-checker-service Bot commented Apr 16, 2026

💚 CLA has been signed

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-presentation-team (Team:obs-presentation)

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-ai-team (Team:obs-ai)

@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@macroscopeapp
Copy link
Copy Markdown
Contributor

macroscopeapp Bot commented Apr 16, 2026

Approvability

Verdict: Needs human review

Pure typo fix changing 'occured' to 'occurred' in error messages and comments across 7 files with no logic changes. However, the author does not own any of the modified files, and changes touch multiple team domains including a security plugin file, warranting brief human acknowledgment from relevant code owners.

You can customize Macroscope's approvability policy. Learn more.

Copy link
Copy Markdown
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

kibana-presentation changes LGTM
code review only

Copy link
Copy Markdown
Contributor

@SoniaSanzV SoniaSanzV left a comment

Choose a reason for hiding this comment

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

thank you for taking the time of fixing this
ingest_pipelines changes LGTM (code review only)

Copy link
Copy Markdown
Contributor

@rmyz rmyz left a comment

Choose a reason for hiding this comment

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

x-pack/solutions/observability/plugins/apm/public/components/shared/error_state_prompt.tsx change LGTM

Copy link
Copy Markdown
Contributor

@kc13greiner kc13greiner left a comment

Choose a reason for hiding this comment

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

Security comment updates LGTM!

@SoniaSanzV SoniaSanzV added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting labels Apr 23, 2026
@SAY-5
Copy link
Copy Markdown
Author

SAY-5 commented Apr 28, 2026

Closing — typo-only PR. Multiple maintainers across the OSS ecosystem have flagged my recent typo-sweep PRs as AI-generated spam (notably hashicorp/nomad#27855, hashicorp/hcl#794, argo-cd reviewers). I should have caught this pattern sooner. Apologies for the noise.

@SAY-5 SAY-5 closed this Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting ci:project-deploy-observability Create an Observability project 💝community release_note:skip Skip the PR/issue when compiling release notes Team:obs-ai Observability AI team Team:obs-presentation Focus: APM UI, Infra UI, Hosts UI, Universal Profiling, Obs Overview and left Navigation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants