Skip to content
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

Implementation - Threema Connector #2

Merged
merged 77 commits into from
Oct 20, 2023
Merged

Implementation - Threema Connector #2

merged 77 commits into from
Oct 20, 2023

Conversation

ivy-fhe
Copy link
Contributor

@ivy-fhe ivy-fhe commented Oct 20, 2023

Initial Implementation of the Threema Connector, passing all builds.

threema-connector/pom.xml Outdated Show resolved Hide resolved
Copy link
Member

@ivy-rew ivy-rew left a comment

Choose a reason for hiding this comment

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

Very nice stuff; some minor things to adapt ... se my comments.
Feel free to ask back if my comments are confusing.

removed linebreak
@ivy-rew
Copy link
Member

ivy-rew commented Oct 20, 2023

one thing I didn't see in the initial review. the product.json must be implemented ... just delete the parts that you don't need and replace placeholders with valid group+artifactIds.
you may use vscode for editing ... as it helps you by keeping the json compliant to the schema and shows help-texts where known.

@ivy-rew ivy-rew merged commit f71d874 into axonivy-market:master Oct 20, 2023
1 check passed
ivy-fhe pushed a commit that referenced this pull request Nov 2, 2023
Implementation - Threema Connector
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.

2 participants