plc4x icon indicating copy to clipboard operation
plc4x copied to clipboard
verified publisher icon apache

S7szl

Open glcj opened this issue 4 years ago • 6 comments

Add the possibility of:

  1. Subscribe to Alarm8.
  2. Recognition of events.
  3. Alarm text processing.
  4. Subscription to cyclical values.
  5. Diagnostic Record Readings (SZL).
  6. Proposal for documentation.

glcj avatar Nov 17 '21 05:11 glcj

LGTM ... only thing I didn't quite understand is why a subscription field has multiple fields inside.

chrisdutz avatar Dec 04 '21 12:12 chrisdutz

I try to resolve the conflicts

sruehl avatar Dec 04 '21 12:12 sruehl

ok might be easier to rebase this... Merge was bit hard an I aborted it due to time constraints...

sruehl avatar Dec 04 '21 13:12 sruehl

Hello, I do not see the changes in the development branch, is this PR for revision? Grateful for your time,

glcj avatar Dec 06 '21 16:12 glcj

no I tried the rebase locally and noticed that it might be easier to rebase the changes onto current develop. Due to timeconstraints I couldn't finish it in time. I try to finish it this weekend.

sruehl avatar Dec 06 '21 16:12 sruehl

@glcj I propose to close this PR and you reopen a new one based on a branch in this repo (feature/s7_subscription. There you need to apply your changes manually and we try then to get them merged ASAP.

sruehl avatar Mar 31 '22 10:03 sruehl