Skip to content

feat: Some random modifications - #31

Merged
tomhula merged 3 commits into
mainfrom
modifications
Aug 1, 2026
Merged

feat: Some random modifications#31
tomhula merged 3 commits into
mainfrom
modifications

Conversation

@jzitnik-dev

@jzitnik-dev jzitnik-dev commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

Small modifications that aren't worth multiple PR's.

  1. Include ktor engine on darwin build.
    Darwin build is a self-container binary so Ktor's client has to be linked, and can't be access from runtime classpath like in android.
  2. Grade now for teacher uses TeacherReference instead of Name.
    It doesn't make sense to use Name which is name of a subject as explained in javadoc. (I was thinking about renaming Name to SubjectName but decided not to just to not break existing usage)
  3. Teacher object now has full RoomReference and not just room name.
    Self-explanatory

On iOS a Kotlin/Native framework is self-contained static binary, so the
HTTP engine must be linked into it at build time, whereas on Android/JVM
the engine is resolved from the app's runtime classpath. Thus this change.
@tomhula
tomhula merged commit f38fa2d into main Aug 1, 2026
2 checks passed
@jzitnik-dev
jzitnik-dev deleted the modifications branch August 1, 2026 19:35
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