Skip to content

feat: Chat bubbles - Load real messages - WPB-19328 #3439

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

Open
wants to merge 9 commits into
base: feature/chat-bubbles-datasource-WPB-19058
Choose a base branch
from

Conversation

dmitrysimkin
Copy link
Contributor

@dmitrysimkin dmitrysimkin commented Aug 7, 2025

TaskWPB-19328 [iOS] Load real messages into new DataSource

Issue

  • Created use case and repository to load conversation messages from DB
  • LoadConversationMessagesRepository was implemented in main app because we can't for now use our core data models in SP packages
  • Load real messages through new datasource
  • Make WireMessagingAssembly be only one entry point to conversation screen functionality, fixed access modificators where needed

Testing

Turn on FF to use chat bubles and open conversation screen

Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

UI accessibility checklist

If your PR includes UI changes, please utilize this checklist:

  • Make sure you use the API for UI elements that support large fonts.
  • All colors are taken from WireDesign.ColorTheme or constructed using WireDesign.BaseColorPalette.
  • New UI elements have Accessibility strings for VoiceOver.

Copy link
Contributor

github-actions bot commented Aug 7, 2025

Test Results

1 951 tests   1 924 ✅  2m 13s ⏱️
  321 suites     27 💤
    2 files        0 ❌

Results for commit 14b08b7.

♻️ This comment has been updated with latest results.

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.

1 participant