Skip to content

ref(explore): remove frontend schema hints code#117717

Merged
JoshuaKGoldberg merged 5 commits into
masterfrom
remove-schema-hints
Jun 18, 2026
Merged

ref(explore): remove frontend schema hints code#117717
JoshuaKGoldberg merged 5 commits into
masterfrom
remove-schema-hints

Conversation

@JoshuaKGoldberg

@JoshuaKGoldberg JoshuaKGoldberg commented Jun 15, 2026

Copy link
Copy Markdown
Member

🔪🔪🔪 code removal! Now that the full schema hints removal flag is rolled out to GA (https://github.com/getsentry/sentry-options-automator/pull/8249), this removes the frontend schema hints code.

Followup PR to remove the backend feature flag registration: #117798

Does most of LOGS-789.

@linear-code

linear-code Bot commented Jun 15, 2026

Copy link
Copy Markdown

LOGS-789

@github-actions github-actions Bot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels Jun 15, 2026
@github-actions

This comment was marked as outdated.

@JoshuaKGoldberg JoshuaKGoldberg changed the title feat(explore): remove schema hints altogether ref(explore): remove schema hints altogether Jun 16, 2026
@JoshuaKGoldberg JoshuaKGoldberg changed the title ref(explore): remove schema hints altogether ref(explore): remove frontend schema hints code Jun 16, 2026
# Conflicts:
#	static/app/views/explore/conversations/overview.tsx
Removes the orphaned 'schema hints' describe block in spansTab.spec.tsx
(its 'See full list' assertion no longer holds after the hints UI was
removed) and the now-unemitted schema_hints_click / schema_hints_drawer
analytics events.
@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as ready for review June 16, 2026 14:45
@JoshuaKGoldberg JoshuaKGoldberg requested review from a team as code owners June 16, 2026 14:45

@nikkikapadia nikkikapadia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

rip my baby (also a couple of comments)

Comment on lines -13 to -22
const COMMON_HINT_KEYS = [
SpanFields.IS_TRANSACTION,
SpanFields.SPAN_OP,
SpanFields.SPAN_DESCRIPTION,
SpanFields.SPAN_DURATION,
SpanFields.TRANSACTION,
FieldKey.HTTP_STATUS_CODE,
SpanFields.RELEASE,
'url',
];

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

i wonder if it's worth putting in a suggested/recommended section in the search bar dropdown since we're getting rid of the common ones and they're buried within the other attributes 🤔 not really blocking just a suggestion for down the line

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah I like that! I actually am unclear on (haven't dug into) what the logic is for generating the suggested ones beyond recents...

Comment thread static/app/views/explore/spans/spansTabSearchSection.spec.tsx Outdated
@JoshuaKGoldberg JoshuaKGoldberg merged commit b4c0ee3 into master Jun 18, 2026
78 checks passed
@JoshuaKGoldberg JoshuaKGoldberg deleted the remove-schema-hints branch June 18, 2026 14:40
JoshuaKGoldberg added a commit that referenced this pull request Jun 18, 2026
Backend half of #117717. Removes the
`organizations:explore-schema-hints-removal` feature flag registration
now that the schema hints UI is being removed altogether.

Closes LOGS-789.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants