Skip to content

Conversation

@SalehTZ
Copy link

@SalehTZ SalehTZ commented May 21, 2025

fixes #33 and resolves #35

Summary

This PR includes several improvements, updates, and fixes across the project and example app.

Changes

  • Documentation

    • Updated the README.md with new configuration notes and a troubleshooting section. (commit, commit)
  • Project Configuration

    • Updated the SDK constraint in pubspec.yaml to ^3.7.2 and added flutter_lints as a dev dependency.
    • Updated .gitignore to include the example/android/app/.cxx/ directory. (commit)
  • Example App

    • Updated dependencies: SDK constraint set to ^3.7.2, cupertino_icons to ^1.0.8, and added flutter_lints ^5.0.0. (commit)
    • Re-enabled the default Flutter test in the example app. (commit)
    • Refactored and simplified the example app’s UI, improved connection status messages, improved widget structure, and added a snackbar for successful subscriptions. (commit)
  • Build System

    • Migrated the example app’s Gradle scripts from Groovy to Kotlin DSL for improved type safety and better IDE support. (commit)
    • Updated Gradle, Kotlin, and plugin versions for compatibility with newer Flutter versions. (commit)

For a full list of recent commits, see the commit history.

SalehTZ and others added 8 commits May 21, 2025 11:47
- Gradle to `8.10.2`
- `ext.kotlin_version` to `1.8.22`
- `com.android.tools.build:gradle` to `8.7.0`
- `JavaVersion` to `JavaVersion.VERSION_11`
This commit migrates the example application's Gradle build scripts from Groovy to Kotlin DSL. This includes updating `build.gradle`, `app/build.gradle`, and `settings.gradle` to their Kotlin equivalents (`.kts`).

The Kotlin DSL offers improved type safety, better IDE support, and a more concise syntax.

Key changes:
- Renamed `.gradle` files to `.gradle.kts`.
- Updated syntax for plugin application, dependencies, and configurations.
- Adjusted Java and Kotlin compilation options (e.g., `JavaVersion.VERSION_1_11`).
- Ensured compatibility with existing project structure and Flutter integration.
- Updated `com.android.application` plugin version to `8.7.0` and `org.jetbrains.kotlin.android` to `1.8.22`.
This commit refactors the example application for `flutter_poolakey`:

- **UI Simplification:** Removed the `title` property from `MyHomePage` and simplified the UI by removing unnecessary comments and whitespace.
- **Connection Status:** Updated the connection status messages to be more concise (e.g., "Service: Connected" to "Connected").
- **Widget Structure:** Minor adjustments to widget nesting and spacing for better readability.
- **Snackbar Messages:** Added a snackbar message for the `subscribeProduct` success case.
The default flutter test was disabled in the example app. This commit re-enables it.
Updated the SDK constraint to ^3.7.2, cupertino_icons to ^1.0.8, and added flutter_lints ^5.0.0 in the example app's pubspec.yaml.
This commit updates the SDK constraint in `pubspec.yaml` to `^3.7.2` and adds `flutter_lints` as a dev dependency. Additionally, it updates the `.gitignore` file to ignore the `example/android/app/.cxx/` directory.
@SalehTZ
Copy link
Author

SalehTZ commented Jun 3, 2025

@nasrabadiAM @mbfakourii Please review and merge if there's no issue.

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.

BuildConfig

2 participants