Skip to content

fix: wants vs has#1017

Merged
mdydek merged 2 commits intofix/1004-invalid-engine-state-on-recording-or-playback-startfrom
fix/1004-invalid-engine-state-on-recording-or-playback-start-locks
Apr 14, 2026
Merged

fix: wants vs has#1017
mdydek merged 2 commits intofix/1004-invalid-engine-state-on-recording-or-playback-startfrom
fix/1004-invalid-engine-state-on-recording-or-playback-start-locks

Conversation

@michalsek
Copy link
Copy Markdown
Member

Closes #

⚠️ Breaking changes ⚠️

Introduced changes

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added/Conducted relevant tests
  • Performed self-review of the code
  • Updated Web Audio API coverage
  • Added support for web
  • Updated old arch android spec file

@mdydek mdydek force-pushed the fix/1004-invalid-engine-state-on-recording-or-playback-start-locks branch from 8dcbdd7 to aea7df4 Compare April 13, 2026 13:02
@mdydek mdydek marked this pull request as ready for review April 14, 2026 07:53
@mdydek mdydek merged commit 5516121 into fix/1004-invalid-engine-state-on-recording-or-playback-start Apr 14, 2026
4 of 6 checks passed
maciejmakowski2003 added a commit that referenced this pull request Apr 14, 2026
* fix: sloppified fix for engine/session management

* fix: format with valid clang-format version

* fix: cleanup hallucination v1

* fix: recorder host object missing function definition

* fix: invalid engine state after session deactivation

* fix: cleanup unnecessary code

* fix: cleanup unnecessary code

* feat: ios engine test coverage + small cleanups

* feat: works

* fix: re-create engine together with nodes

* fix: remove tmp files, better recorder start error flow

* fix: cleanup tests

* fix: cleanup stress tests slop

* chore: cleanup

* ci: format

* fix: ios recorder tests

* fix: wants vs has (#1017)

* fix: removed unnecesary screen

* test: recorder restarts an engine

* refactor: removed old todos and use resolved buffer size

* fix: removed batchDurationSeconds

---------

Co-authored-by: michal <dydmichal@gmail.com>
Co-authored-by: maciejmakowski2003 <maciejmakowski2003@users.noreply.github.com>
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