Skip to content

feat: Define app.screen.name#2744

Merged
lmolkova merged 6 commits intoopen-telemetry:mainfrom
limhjgrace:feat/screen-name
Sep 23, 2025
Merged

feat: Define app.screen.name#2744
lmolkova merged 6 commits intoopen-telemetry:mainfrom
limhjgrace:feat/screen-name

Conversation

@limhjgrace
Copy link
Copy Markdown
Contributor

@limhjgrace limhjgrace commented Sep 7, 2025

Fixes #2743

Revisions

  • Rev2: Rebased off of mainline, updated notes and examples to address comments (diff).
  • Rev3: Fixed linting errors that were causing some of the workflows to fail (diff).
  • Rev4: Updated note for app.screen.name to address comments/SIG discussion, added app.screen.id (diff)

Changes

There is currently no semantic convention for capturing application screen names. This PR proposes app.screen.name, as discussed during Client SIG on 07/22/2025.

Merge requirement checklist

  • CONTRIBUTING.md guidelines followed.
  • Change log entry added, according to the guidelines in When to add a changelog entry.
    • If your PR does not need a change log, start the PR title with [chore]
  • Links to the prototypes or existing instrumentations (when adding or changing conventions)

@limhjgrace limhjgrace requested review from a team as code owners September 7, 2025 15:56
@github-actions github-actions Bot added the enhancement New feature or request label Sep 7, 2025
@limhjgrace limhjgrace changed the title feat: Define app.screen.name feat: Define app.screen.name Sep 7, 2025
@limhjgrace limhjgrace changed the title feat: Define app.screen.name feat: Define app.screen.name Sep 7, 2025
@lmolkova lmolkova moved this from Untriaged to Awaiting codeowners approval in Semantic Conventions Triage Sep 7, 2025
Comment thread model/app/registry.yaml Outdated
Comment thread model/app/events.yaml
@limhjgrace limhjgrace reopened this Sep 9, 2025
Comment thread model/app/registry.yaml Outdated
@lmolkova lmolkova moved this from Awaiting codeowners approval to Needs More Approval in Semantic Conventions Triage Sep 11, 2025
Comment thread model/app/registry.yaml
Comment thread docs/registry/attributes/app.md Outdated
Copy link
Copy Markdown
Contributor

@breedx-splk breedx-splk left a comment

Choose a reason for hiding this comment

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

LGTM thank you!

@lmolkova lmolkova moved this from Needs More Approval to Ready to be Merged in Semantic Conventions Triage Sep 22, 2025
@github-project-automation github-project-automation Bot moved this from Ready to be Merged to Needs More Approval in Semantic Conventions Triage Sep 23, 2025
@lmolkova lmolkova added this pull request to the merge queue Sep 23, 2025
Merged via the queue into open-telemetry:main with commit 4658d9b Sep 23, 2025
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:app enhancement New feature or request

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Add app.screen.name

8 participants