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

feat!: migrated plugin to flutter 3.19 template #332

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

OutdatedGuy
Copy link

@OutdatedGuy OutdatedGuy commented Jun 13, 2023

⚠️ Might have Breaking Changes

Changes

  • Updated .gitignore files according to new template
  • Unchecked local config(which should not be version controlled) files from version control (About 100 to 150 files)
  • Added new template code analysis files
  • Updated android build.gradle template
  • Added support for AGP 8
  • Updated whole example app to latest Flutter 3.19 Template
  • Refactored and cleaned example app main.dart file code
  • Updated code to follow Dart 3 specifications
  • Removed unnecessary/deprecated flutter.jar file
  • Updated dependencies to latest
  • Replaced deprecated code with new recommended ones

Demo

Below is a Screen recording with actual successful payment on Android and iOS devices. Also checks event handler for payment cancel.

New.Razorpay.Plugin.Demo.mp4

@OutdatedGuy OutdatedGuy force-pushed the update-to-modern-template branch from 405d3ea to 1384198 Compare June 13, 2023 09:49
@OutdatedGuy OutdatedGuy marked this pull request as ready for review June 13, 2023 10:04
@OutdatedGuy
Copy link
Author

@ramprasadAnand @razorpay Can you please have a look here and tell me if it is good or any changes are required.

@OutdatedGuy OutdatedGuy force-pushed the update-to-modern-template branch from 74126c7 to 29f4896 Compare March 28, 2024 12:37
@OutdatedGuy OutdatedGuy changed the title feat!: migrated plugin to flutter 3.10 template feat!: migrated plugin to flutter 3.19 template Mar 28, 2024
@OutdatedGuy
Copy link
Author

OutdatedGuy commented Mar 28, 2024

@shibu000 @pshibu000 @vivekshindhe Can you please have a look here and tell me if it is good or any changes are required.

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