Skip to content

Feat: add client version tracking and callback to TaskControllerServer#693

Merged
ad3154 merged 3 commits into
Open-Agriculture:mainfrom
gunicsba:feature/tc-server-version-tracking
Jun 6, 2026
Merged

Feat: add client version tracking and callback to TaskControllerServer#693
ad3154 merged 3 commits into
Open-Agriculture:mainfrom
gunicsba:feature/tc-server-version-tracking

Conversation

@gunicsba

Copy link
Copy Markdown
Contributor

Add automatic version detection and callback mechanism for Task Controller server to track connected client versions.

Changes:

  • Add on_client_version_received() virtual callback for applications
  • Add get_client_version() getter to retrieve stored client version
  • Add request_client_version() method for proactive version queries
  • Automatically request version when new client detected via WorkingSetMaster
  • Add INFO/DEBUG logging for version exchange events
  • Add comprehensive unit tests for version tracking functionality

The callback fires when client version info is received, supporting both V3 (SecondEditionDraft) and V4+ (SecondPublishedEdition). Version exchange is optional in V3 but automatic in the server implementation.

Add automatic version detection and callback mechanism for Task Controller
server to track connected client versions.

Changes:
- Add on_client_version_received() virtual callback for applications
- Add get_client_version() getter to retrieve stored client version
- Add request_client_version() method for proactive version queries
- Automatically request version when new client detected via WorkingSetMaster
- Add INFO/DEBUG logging for version exchange events
- Add comprehensive unit tests for version tracking functionality

The callback fires when client version info is received, supporting both
V3 (SecondEditionDraft) and V4+ (SecondPublishedEdition). Version exchange
is optional in V3 but automatic in the server implementation.
@ad3154

ad3154 commented Jun 1, 2026

Copy link
Copy Markdown
Member

This looks OK to me. If you can update the example to fix the pipeline failure, then I would be OK with approving it.

@ad3154 ad3154 self-assigned this Jun 1, 2026
@ad3154 ad3154 added enhancement New feature or request iso: task controller Related to the ISO-11783:10 standard labels Jun 1, 2026
@gunicsba

gunicsba commented Jun 6, 2026

Copy link
Copy Markdown
Contributor Author

I tested this with a couple equipments this week and seems to work with all of them .

@ad3154
ad3154 merged commit 5602992 into Open-Agriculture:main Jun 6, 2026
10 checks passed
@gunicsba
gunicsba deleted the feature/tc-server-version-tracking branch June 6, 2026 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request iso: task controller Related to the ISO-11783:10 standard

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants