Skip to content

Conversation

bbaarriiss
Copy link
Contributor

@bbaarriiss bbaarriiss commented Aug 19, 2025

This PR contains a sample code (using the SDK) written in Java language.

  • Extend WireEventsHandlerDefault
  • Reply the message with the same content with a minimal addition to the end.

PR Submission Checklist for internal contributors

  • The PR Title

    • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
    • contains a reference JIRA issue number like SQPIT-764
    • answers the question: If merged, this PR will: ... ³
  • The PR Description

    • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

What's new in this PR?

Issues

This PR contains a sample code (using the SDK) written in Java language.


References
  1. https://sparkbox.com/foundry/semantic_commit_messages
  2. https://github.com/wireapp/.github#usage
  3. E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

Extend WireEventsHandlerDefault
Reply the message with the same content with a minimal addition to the end.
@bbaarriiss bbaarriiss self-assigned this Aug 19, 2025
@bbaarriiss bbaarriiss requested a review from a team as a code owner August 19, 2025 15:33
@CLAassistant
Copy link

CLAassistant commented Aug 19, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@alexandreferris alexandreferris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good and clean, just got one strange comment 👀

Remove ChatService and simplify the flow.
Use Java 17 instead of 21
@bbaarriiss bbaarriiss merged commit bec3cb1 into main Aug 20, 2025
2 checks passed
Copy link
Contributor

@MarianKijewski MarianKijewski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the simplification caused by moving ChatService logic into Events handler

@bbaarriiss bbaarriiss deleted the feat/create-sample-for-java_WPB-19280 branch August 20, 2025 14:36
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.

5 participants