Skip to content

feat: add GetConversationEpochFromCC use case and integrate into debug conversation screen#4038

Open
MohamadJaara wants to merge 1 commit intodevelopfrom
mo/feat/add-cc-epoch-to-the-inspect-conv-screen
Open

feat: add GetConversationEpochFromCC use case and integrate into debug conversation screen#4038
MohamadJaara wants to merge 1 commit intodevelopfrom
mo/feat/add-cc-epoch-to-the-inspect-conv-screen

Conversation

@MohamadJaara
Copy link
Copy Markdown
Member

What's new in this PR?

Issues

The debug inspect conversation screen showed the epoch from the local conversation object, which is not reliable enough for debugging.

Solutions

  • Kept the existing local epoch visible and marked it as deprecated.
  • Added a new debug use case to fetch the conversation epoch from CC.
  • Added a small refresh action because the CC epoch is fetched on demand and not exposed as a flow.
  • Displayed the CC epoch as a separate line in the inspect conversation screen.

Testing

Test Coverage

  • I have added automated test to this contribution

How to Test

  • Verified the new Kalium use case with unit tests.
  • Verified the debug conversation ViewModel with unit tests.
  • Ran static analysis successfully.

Notes

The CC epoch is only available for MLS conversations.

@sonarqubecloud
Copy link
Copy Markdown

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant