Skip to content

Sync from internal repo (2024/06/25) #101

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

Merged
merged 3 commits into from
Jun 25, 2024

Conversation

Swimburger
Copy link
Contributor

No description provided.

GitOrigin-RevId: 29cbcf7a15d72ca261f3f2a9f2592378474a859c
@Swimburger Swimburger changed the title Sync from internal repo (YYYY/MM/DD) Sync from internal repo (2024/06/25) Jun 25, 2024
Copy link

Comment on lines +26 to +29
# This tag is used as the Fern package name for the channel and messages, even tho the tag is not linked to anything.
# x-fern-group-sdk-name also doesn't work on channels and messages.
# Hence, it has to be "realtime" and can't be "streaming" until these gaps in Fern are fixed.
- name: realtime
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dsinghvi FYI, the fern definition is using the first tag, even tho it's not referenced anywhere, to determine the package of the name.
It would be great if we can have a more explicit way to control the package name.

Comment on lines +27 to +30
- section: Streaming
contents:
- realtime/createTemporaryToken
- realtime/realtime
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was able to keep having realtime as the package name, while still using streaming in the docs by using a section here, and manually specifying the contents of the section.

Copy link

Copy link

@Swimburger Swimburger merged commit 7b06b4d into main Jun 25, 2024
5 checks passed
@Swimburger Swimburger deleted the E2B031BE6A11C67EA5D87E19B5EDAE2E branch June 25, 2024 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant