feature/BA-2601-group-admin-updates

[rf2tsl]

Updating group chat to include existing participant ids, the purpose is exclude the existing participant ids from being selected on the frontend app

Summary by CodeRabbit

• New Features
  • Chat rooms now expose a participant IDs field that provides direct access to a list of all participant identifiers, improving query efficiency and simplifying access to participant information through the API.

[coderabbitai]

Walkthrough

A new participant_ids GraphQL field has been added to BaseChatRoomObjectType with an associated resolver that retrieves and exposes the relay IDs of all participants in a chat room.

Changes

Cohort / File(s)	Summary
GraphQL Participant IDs Field baseapp_chats/graphql/object_types.py	Added participant_ids GraphQL field to BaseChatRoomObjectType with resolver method that computes relay IDs from participant profile IDs and exposes them in the schema.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• feature/BA-2703-update-group-chat-participants #357: Modifies BaseChatRoomObjectType to expose participant-related data through schema changes.

Suggested reviewers

• nossila
• Hercilio1
• vitorguima

Poem

🐰 A list of IDs, now crystal and clear,
Chat participants unveiled without fear,
Relay paths sparkle with participant names,
GraphQL relays bring connection and games! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	The title 'feature/BA-2601-group-admin-updates' is a branch name convention, not a descriptive pull request title that summarizes the actual change (exposing participant IDs in GraphQL schema).	Change the title to something descriptive like 'Add participant_ids field to BaseChatRoomObjectType' or 'Expose group chat participant IDs in GraphQL schema to enable frontend filtering'.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/BA-2601-group-admin-updates

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

baseapp_chats/graphql/object_types.py (1)

13-13: Nit: add trailing comma for consistency.

The other items in this import block use trailing commas. Adding one here keeps diffs cleaner if more imports are added later.

-    get_obj_relay_id
+    get_obj_relay_id,

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}