Skip to content

Move PropertyImpl methods to cpp - #1276

Open
denise-opendaq wants to merge 3 commits into
mainfrom
move_propertyImpl_to_cpp
Open

Move PropertyImpl methods to cpp #1276
denise-opendaq wants to merge 3 commits into
mainfrom
move_propertyImpl_to_cpp

Conversation

@denise-opendaq

@denise-opendaq denise-opendaq commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Brief

Move PropertyImpl methods to cpp part

Description

  • move PropertyImpl methods to cpp part
  • remove ConfigClientPropertyImpl class and use pure Property for ConfigClient
  • in test_native_device_modules.cpp replace LATEST_CONFIG_PROTOCOL_VERSION with config_protocol::GetLatestConfigProtocolVersion() so developer doesn't need to manually update the version for test while bumping config version

API changes

+ IPropertyInternal::getOwner(IPropertyObject** owner)

Required module changes

There is PR for the updated opcua client property (openDAQ/OpcUaModules#35). but the older compiled module should work as well

@denise-opendaq denise-opendaq self-assigned this Aug 20, 2026
@denise-opendaq denise-opendaq changed the title Move PropertyImpl method to cpp Move PropertyImpl methods to cpp Aug 20, 2026
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.

1 participant