Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/rework text sender #140

Merged
merged 19 commits into from
Mar 6, 2025
Merged

Feature/rework text sender #140

merged 19 commits into from
Mar 6, 2025

Conversation

kaleidot725
Copy link
Owner

@kaleidot725 kaleidot725 commented Mar 6, 2025

This pull request includes several updates to the project, focusing on dependency management, MVI architecture implementation, and enhancements to the MainStateHolder and related UI components.

Dependency Management Updates:

  • Updated compose_hot_reload version to 1.0.0-alpha01 in gradle/libs.versions.toml and adjusted plugin references accordingly. [1] [2]
  • Removed obsolete Maven repositories from settings.gradle.kts. [1] [2]

MVI Architecture Implementation:

  • Introduced MVI, MVIAction, MVISideEffect, MVIState, and MVIDelegate interfaces and classes to support MVI architecture. [1] [2] [3] [4] [5]

MainStateHolder Enhancements:

  • Added setup, refresh, and dispose methods for textCommandStateHolder and screenshotStateHolder in MainStateHolder. [1] [2]
  • Simplified TextCommandScreen and ScreenshotScreen by utilizing onAction methods and ScreenshotAction enum. [1] [2]

Other Notable Changes:

  • Updated the download link in README.md to point to the latest release version.
  • Added Serializable annotation to TextCommand and included an id field with UUID generation.
  • Removed unused use cases from DomainModule. [1] [2] [3] [4]

Copy link

github-actions bot commented Mar 6, 2025

1 Warning
⚠️ Big PR

Generated by 🚫 Danger

@kaleidot725 kaleidot725 merged commit ccc6704 into main Mar 6, 2025
2 checks passed
@kaleidot725 kaleidot725 deleted the feature/rework_text_sender branch March 6, 2025 12:42
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