Skip to content

Add more users in demo-app - #1806

Merged
PratimMallick merged 1 commit into
developfrom
demo-app/rahullohra/add-more-users
Sep 4, 2026
Merged

Add more users in demo-app#1806
PratimMallick merged 1 commit into
developfrom
demo-app/rahullohra/add-more-users

Conversation

@rahul-lohra

@rahul-lohra rahul-lohra commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Goal

Closes: #AND-1487
Add more users in demo-app

Implementation

Add more users in demo-app

🎨 UI Changes

None

Testing

Smoke test ringing

Summary by CodeRabbit

  • New Features
    • Added five built-in demo users with profile images:
      • Guilherme Bier
      • Alexey Alter-Pesotskiy
      • Petar Velikov
      • Gulzar Ahmed
      • Bruno Ribeiro

@rahul-lohra rahul-lohra self-assigned this Sep 4, 2026
@rahul-lohra
rahul-lohra requested a review from a team as a code owner September 4, 2026 08:38
@rahul-lohra rahul-lohra added the pr:demo-app Changes specific to demo app label Sep 4, 2026
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

PR checklist ✅

All required conditions are satisfied:

  • Title length is OK (or ignored by label).
  • At least one pr: label exists.
  • Sections ### Goal, ### Implementation, and ### Testing are filled, or the PR is bot-authored.
  • An issue is linked (Linear ticket or GitHub issue), or the PR is bot-authored.

🎉 Great job! This PR is ready for review.

@rahul-lohra
rahul-lohra force-pushed the demo-app/rahullohra/add-more-users branch from 00dfba3 to 36e4732 Compare September 4, 2026 08:40
@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Team

Run ID: 69fba53f-9bfe-4cc5-88bc-d488b7efb56d

📥 Commits

Reviewing files that changed from the base of the PR and between 01799d5 and 36e4732.

📒 Files selected for processing (1)
  • demo-app/src/main/kotlin/io/getstream/video/android/models/Users.kt

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.


Walkthrough

The demo application now includes five additional built-in users with IDs, names, roles, and Slack profile images.

Changes

Demo user catalog

Layer / File(s) Summary
Add built-in user entries
demo-app/src/main/kotlin/io/getstream/video/android/models/Users.kt
builtInUsers() now defines five users with IDs, names, roles, and Slack image URLs.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 36e47

This change adds five built-in demo users and their avatar URLs. No current merge-readiness risk remains.

Suggested reviewers: aleksandar-apostolov

Poem

A rabbit checks the user list bright
Five new faces join the night
Names and roles sit neat in rows
Slack-born portraits softly glow
The demo burrow hops along

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description check ✅ Passed The description states the goal, links issue AND-1487, summarizes the implementation, confirms no UI changes, and describes the ringing smoke test. The checklist and GIF sections are not included, but…
Title check ✅ Passed The title clearly identifies the main change: adding more users to the demo app.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch demo-app/rahullohra/add-more-users

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

Comment @coderabbitai help to get the list of available commands.

@rahul-lohra rahul-lohra changed the title [AND-1487] Add more users in demo-app Add more users in demo-app Sep 4, 2026
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

SDK Size Comparison 📏

SDK Before After Difference Status
stream-video-android-core 12.30 MB 12.30 MB 0.00 MB 🟢
stream-video-android-ui-xml 5.70 MB 5.70 MB 0.00 MB 🟢
stream-video-android-ui-compose 6.23 MB 6.23 MB 0.00 MB 🟢

@sonarqubecloud

sonarqubecloud Bot commented Sep 4, 2026

Copy link
Copy Markdown

@PratimMallick
PratimMallick merged commit 083e929 into develop Sep 4, 2026
19 checks passed
@PratimMallick
PratimMallick deleted the demo-app/rahullohra/add-more-users branch September 4, 2026 09:12
@stream-public-bot stream-public-bot added the released Included in a release label Sep 11, 2026
@stream-public-bot

Copy link
Copy Markdown
Collaborator

🚀 Available in v1.33.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:demo-app Changes specific to demo app released Included in a release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants