Skip to content

Conversation

@alon-getrise
Copy link
Contributor

Proposed changes

Merge timing

Related issues (optional)

Updated event names based on requirements for Proper Case
@alon-getrise alon-getrise requested a review from a team as a code owner November 3, 2025 17:40
@alon-getrise alon-getrise requested a review from pwseg November 3, 2025 17:40
@netlify
Copy link

netlify bot commented Nov 3, 2025

Deploy Preview for segment-docs ready!

Name Link
🔨 Latest commit dded514
🔍 Latest deploy log https://app.netlify.com/projects/segment-docs/deploys/6908e90227f9980008342ff4
😎 Deploy Preview https://deploy-preview-8128--segment-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@dineshraj-arunmozhi
Copy link
Contributor

@pwseg This change was suggested by our team(Partner support), since the event didn't match the proper case as per the template.

@pwseg pwseg changed the title Rise ai/source Update Rise AI source event names Nov 7, 2025
@pwseg
Copy link
Contributor

pwseg commented Nov 7, 2025

@dineshraj-arunmozhi @alon-getrise Thanks, I'll get this merged. But are these the actual event names Rise AI is sending to Segment? like they show up like that in the payloads and in the warehouse tables? If so, they should stay lowercase and be formatted as inline code. Like walkthrough-progress and chats.

Capitalizing them might make it seem like those are display names instead of the raw event names, which could cause confusion for users checking their event data or schemas.

I'm going to merge this for now, but let me know if Rise uses title-cased labels in the UI and you’d prefer to surface those instead, just curious as to where users will actually see those event names.

@pwseg pwseg added the content-update updates to content that are not new features, includes grammar fixes, added notes label Nov 7, 2025
@pwseg pwseg merged commit cc14a0d into segmentio:develop Nov 7, 2025
4 checks passed
@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

@alon-getrise
Copy link
Contributor Author

alon-getrise commented Nov 8, 2025 via email

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

Labels

content-update updates to content that are not new features, includes grammar fixes, added notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants