Skip to content

Conversation

@filipecabaco
Copy link
Member

I have read the CONTRIBUTING.md file.

YES

What kind of change does this PR introduce?

improve protocol docs; move settings to a guides

@vercel
Copy link

vercel bot commented Jul 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
design-system ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 18, 2025 9:28pm
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 18, 2025 9:28pm
studio-self-hosted ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 18, 2025 9:28pm
studio-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 18, 2025 9:28pm
ui-library ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 18, 2025 9:28pm
zone-www-dot-com ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 18, 2025 9:28pm
2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cms ⬜️ Ignored (Inspect) Jul 18, 2025 9:28pm
studio ⬜️ Ignored (Inspect) Jul 18, 2025 9:28pm

@supabase
Copy link

supabase bot commented Jul 17, 2025

This pull request has been ignored for the connected project xguihxuzqibwxjnimxev because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@filipecabaco filipecabaco force-pushed the realtime/improve-protocol-docs branch 10 times, most recently from 3c87730 to 3c6770d Compare July 18, 2025 00:33
@filipecabaco filipecabaco force-pushed the realtime/improve-protocol-docs branch from 3c6770d to 9a895f4 Compare July 18, 2025 10:16
| Event Type | Description | Client Sent | Server Sent | Requires Ref |
|------------|-------------|--------------|-------------|--------------|
| `phx_join` | Initial message to join a channel and configure features ||||
| `phx_close` | Message from server to signal channel closed ||||
Copy link
Contributor

Choose a reason for hiding this comment

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

This one says "message from server" but "client sent"-only, is that intentional?

Copy link
Member Author

Choose a reason for hiding this comment

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

thank you for catching it

Copy link
Contributor

Choose a reason for hiding this comment

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

I think you refer to this as --dark in the img src?

Copy link
Member Author

Choose a reason for hiding this comment

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

yes forgot to push change 😓

filipecabaco and others added 3 commits July 18, 2025 15:53
Co-authored-by: Charis <26616127+charislam@users.noreply.github.com>
Co-authored-by: Charis <26616127+charislam@users.noreply.github.com>
@filipecabaco
Copy link
Member Author

made the fixes, thank you @charislam 🙏

@w3b6x9 w3b6x9 enabled auto-merge (squash) July 18, 2025 21:14
@w3b6x9 w3b6x9 merged commit 43c0c25 into master Jul 18, 2025
18 checks passed
@w3b6x9 w3b6x9 deleted the realtime/improve-protocol-docs branch July 18, 2025 21:28
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.

3 participants