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

Feat/hop #839

Draft
wants to merge 14 commits into
base: develop
Choose a base branch
from
Draft

Feat/hop #839

wants to merge 14 commits into from

Conversation

glcj
Copy link
Contributor

@glcj glcj commented Mar 11, 2023

Integration with Hop V2.X.X in progress. Action and Transform Read, Write and Events are working, Subscription is missing due to problems in the S7 driver, these routines were removed in version 0.10.0 of Plc4x. The resources for the modification are added to the portal and proceed to their documentation.

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

Trivy found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

@sruehl sruehl marked this pull request as draft May 8, 2023 17:05
… the corrections in the Hop integration for the correct handling of Actions and Transforms.
@chrisdutz
Copy link
Contributor

Your mspec changes shouldn't cause any serious problems ... I'll have a look at if the build works on my machine.

I think you can remove the ".cache" files as I think IntelliJ creates them when using the IDE editor ... same with the generated png files. These should be created during the website generation.

I love your Toddies by the way.

@ottlukas ottlukas added java Pull requests that update Java code feature and removed improvement labels Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature java Pull requests that update Java code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants