Skip to content

Conversation

@borditamas
Copy link
Collaborator

No description provided.

pom.xml Outdated

<repositories>

<repositories> <!-- TODO: delete this after new app library release -->
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

remember to delete!

pom.xml Outdated
<artifactId>arrowhead-application-library-java-spring</artifactId>
<version>4.4.0.0</version>
</dependency>
<dependency> <!-- TODO: delete this after new app library release -->
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

remember to delete!

@borditamas borditamas requested a review from rbocsi August 24, 2021 23:24
Copy link
Contributor

@rbocsi rbocsi left a comment

Choose a reason for hiding this comment

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

see inline comments

@borditamas
Copy link
Collaborator Author

@rbocsi during the implmentation of the working demo, I have implemented a general execution support which I have put here in the skeleton too. Please check

@borditamas borditamas requested a review from rbocsi August 26, 2021 07:54
Copy link
Contributor

@rbocsi rbocsi left a comment

Choose a reason for hiding this comment

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

see inline comments

# Conflicts:
#	application-skeleton-provider/src/main/java/eu/arrowhead/application/skeleton/provider/ProviderApplicationInitListener.java
#	application-skeleton-publisher/src/main/java/eu/arrowhead/application/skeleton/publisher/PublisherApplicationInitListener.java
#	application-skeleton-subscriber/src/main/java/eu/arrowhead/application/skeleton/subscriber/SubscriberApplicationInitListener.java
#	pom.xml
@borditamas
Copy link
Collaborator Author

Status update: Waiting for Core v4.5.0 release (App-lib will contain Choreographer realated DTOs)

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.

3 participants