Skip to content

Comments

[REFACTOR]: Move streamApiLow#120

Open
Doominika wants to merge 20 commits intoandroid-stream-api-testingfrom
refactor/move-stream-api-low
Open

[REFACTOR]: Move streamApiLow#120
Doominika wants to merge 20 commits intoandroid-stream-api-testingfrom
refactor/move-stream-api-low

Conversation

@Doominika
Copy link
Contributor

Description

Move streamApiLow to privmx-endpoint.

@djenczewski djenczewski self-requested a review February 16, 2026 10:49
Copy link
Member

@djenczewski djenczewski left a comment

Choose a reason for hiding this comment

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

  1. StreamApiLow and WebRtcInterface should be inside modules/stream
  2. All model classes should be inside model or model/stream
  3. Remove this classes from android and jvm sources

Copy link
Member

@djenczewski djenczewski left a comment

Choose a reason for hiding this comment

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

  1. this code doesn't compile
  2. Can model/events classes be recursively moved to model/stream/events?


// private native Long init(Connection connection, EventApi eventApi) throws IllegalStateException;

private static native StreamApiLow create(
Copy link
Member

Choose a reason for hiding this comment

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

this is wrong implementation of constructors and create method

… refactor/move-stream-api-low

# Conflicts:
#	jni-wrappers/privmx-endpoint/src/cpp/parsers/model_native_initializers.cpp
#	privmx-endpoint-streams/jvm/src/main/java/com/simplito/java/privmx_endpoint/model/VideoTrackInfo.java
#	privmx-endpoint/src/main/java/com/simplito/java/privmx_endpoint/model/events/eventTypes/StreamEventType.java
#	privmx-endpoint/src/main/java/com/simplito/java/privmx_endpoint/modules/stream/StreamApiLow.java
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.

2 participants