-
Notifications
You must be signed in to change notification settings - Fork 396
Update Rise AI source event names #8128
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Updated event names based on requirements for Proper Case
Update index.md
✅ Deploy Preview for segment-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
@pwseg This change was suggested by our team(Partner support), since the event didn't match the |
|
@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 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. |
|
Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site. |
|
It was a request from your support team
…On Fri, Nov 7, 2025 at 11:40 PM pwseg ***@***.***> wrote:
*pwseg* left a comment (segmentio/segment-docs#8128)
<#8128 (comment)>
@dineshraj-arunmozhi <https://github.com/dineshraj-arunmozhi>
@alon-getrise <https://github.com/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.
—
Reply to this email directly, view it on GitHub
<#8128 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BSL6XW645F2GBPMUEVPZTIL33UGWHAVCNFSM6AAAAACLAKLBISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTKMBVGA4TEMJVG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Proposed changes
Merge timing
Related issues (optional)