Skip to content

Build flutter real time multi user chat app#407

Closed
MahmoudTark wants to merge 5 commits intodirectus:mainfrom
MahmoudTark:Build-Flutter-Real-Time-Multi-user-chat-app
Closed

Build flutter real time multi user chat app#407
MahmoudTark wants to merge 5 commits intodirectus:mainfrom
MahmoudTark:Build-Flutter-Real-Time-Multi-user-chat-app

Conversation

@MahmoudTark
Copy link

Full tutorial of how to Build Flutter Application with real time users chat

@MahmoudTark MahmoudTark requested a review from a team as a code owner July 20, 2025 09:57
@vercel
Copy link

vercel bot commented Jul 20, 2025

@MahmoudTark is attempting to deploy a commit to the Directus Team on Vercel.

A member of the Team first needs to authorize it.

@rijkvanzanten rijkvanzanten requested a review from BB-Loft August 4, 2025 20:04
@rijkvanzanten
Copy link
Member

Hi @MahmoudTark! Thanks for putting this tutorial together :)

Unfortunately I don't think this is quite ready to merge as-is. My main concerns are:

  • sendMessage and loadMessages are left as stubs with "implement here" comments. A reader following along won't end up with a working app
  • The WebSocket subscription will cause duplicate messages: messages are added optimistically and received back via the subscription, without deduplication
  • Code blocks are missing language tags (dart, yaml)
  • KDirectusApiUrl is referenced but never defined, so could cause broken setups if followed exactly
  • The auth setup is contradictory (Public policy with full access, but also token-based WebSocket auth)

I'll close this for now, but would love to see a revised version that has the full working implementation. The Flutter + Realtime combo is a great tutorial topic! 🫶

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.

2 participants

Comments