Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • Briar Desktop Briar Desktop
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 174
    • Issues 174
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 15
    • Merge requests 15
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • briar
  • Briar DesktopBriar Desktop
  • Merge requests
  • !34

Migrate chat view to view model

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Mikolai Gütschow requested to merge 95-conversation-viewmodel into main Oct 27, 2021
  • Overview 56
  • Commits 3
  • Pipelines 7
  • Changes 24

The view model listens to events to update the chat view on the fly when new messages arrive or are sent. Also adds the ability to actually send messages. In the second commit, all the Briar API calls were finally moved out of Composables by slightly rewriting the sidebar logic that was still relying on it (adding yet another view model).

Closes #95 (closed), #69 (closed), #90 (closed)

This PR builds on top of !33 (merged) and should be merged after it. Currently the base branch is set to !33 (merged) for an easier diff.

Edited Nov 01, 2021 by Mikolai Gütschow
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: 95-conversation-viewmodel