Skip to content

feat: update flutter_sms for latest Flutter compatibility #102

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

feat: update flutter_sms for latest Flutter compatibility #102

wants to merge 2 commits into from

Conversation

@verionyxsolutions
Copy link

  • Migrated to Flutter 3.32
  • Converted Android to Kotlin DSL and updated AGP
  • Updated iOS deployment target and cleaned up Podfile
  • Fixed example app
  • Removed deprecated APIs

@verionyxsolutions
feat: update flutter_sms for latest Flutter compatibility
800c23c 
- Migrated to Flutter 3.32
- Converted Android to Kotlin DSL and updated AGP
- Updated iOS deployment target and cleaned up Podfile
- Fixed example app
- Removed deprecated APIs
@verionyxsolutions
Copy link
Author

✨ Summary

This PR updates the flutter_sms plugin to be compatible with the latest Flutter versions and mobile platform requirements.

🔧 Changes Made

  • Migrated plugin to support Flutter 3.32.0
  • Updated Android embedding
  • Ensured compatibility with Android 15+ (API 35)
  • Modernized iOS integration, updated Podspec and iOS platform version
  • Replaced deprecated APIs and removed outdated Gradle/Kotlin warnings
  • Updated the example app for both platforms (Android/iOS)

✅ Testing

  • Tested on Flutter 3.32.0
  • Verified functionality on:
    • Android 13 device (Nokia 2.4)
    • iOS 18.5 real Device (iPhone 14 Pro Max)
  • Example app builds and sends SMS successfully. Direct send sms testing need to be verified yet.

📌 Notes

  • This PR maintains backward compatibility where possible
  • Please review and let me know if additional adjustments are required

Let me know if there's anything I should change or if you'd like separate PRs for Android/iOS updates.

rodydavis
rodydavis requested changes Jun 1, 2025
View reviewed changes
Copy link
Member

@rodydavis rodydavis left a comment

Choose a reason for hiding this comment

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

Added some comments, but overall looks great!

@verionyxsolutions
Implemented author-requested compatibility updates
6f15cd5 
- Replace dart:html with package:web

- Update author info and clean up permissions

- Remove unnecessary namespace

- Add widget tests and update pubspec environment
rodydavis
rodydavis approved these changes Jun 2, 2025
View reviewed changes
Copy link
Member

@rodydavis rodydavis left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rodydavis rodydavis rodydavis approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@verionyxsolutions @rodydavis