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

[dev 4.0.1 -> staging] Add namespace support for Android #107

Merged
merged 7 commits into from
Apr 23, 2024

Conversation

cacheung
Copy link
Contributor

@cacheung cacheung commented Apr 22, 2024

  • Add namespace support for Android Gradle 8.x requirement in plugins
  • Update Flutter SDK support version to 3.x
  • Update compileSdk to 34
  • Update Sample App to use Grade 8.x
  • Update GitHub issue_template
  • Update plugins release version to 4.0.1

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

egonm12 and others added 3 commits April 18, 2024 16:44
* Add conditional namespace to support gradle 8

* Use git as pub resource

* Revert "Use git as pub resource"

This reverts commit 027911d.

* fix

* fix

* fix

* Revert "fix"

This reverts commit 2a5a804.

* Revert "fix"

This reverts commit fbee65b.

* Revert "fix"

This reverts commit 746c0ca.

* Revert "Revert "Use git as pub resource""

This reverts commit 4b7485e.

* Revert "Use git as pub resource"

This reverts commit 027911d.
…s and Update issue_template (#106)

* bump the major version to 4.0.0 (to include Android SDK 3.0.0 and iOS SDK 5.0.0)

* specify the ios version

* update CI to use Xcode 15.0.1

* fix CI syntax issue

* draft

* cleanup

* update API doc

* address review comments

* address a review comment

* correct timeout accuracy

* Update FlutterAEPCorePlugin.m

* feat: ⬆️ messaging api changes

* Update EdgeBridge sample app and readme

Update EdgeBridge sample app and readme

* refactor: ♻️ rename edge event type for messaging

* chore: reenable messaging

* fix: 🐛 fixes for messaging

* disable unit tests

* docs: 📝 remove beta language from messaging docs

* Update README for Privacy Manifest changes (#100)

* update main README to show information about "Privacy Manifest"

* fix: 🐛 fix oncontentloaded to be android only

* Update Flutter version for the plugins, update sample app gradle files and update issue_template

Update Flutter version for the plugins
Update sample app gradle files
Update issue_template

---------

Co-authored-by: yansong <[email protected]>
Co-authored-by: dsoffiantini <[email protected]>
update ci java-version
@yangyansong-adbe
Copy link
Contributor

yangyansong-adbe commented Apr 22, 2024

@cacheung
https://github.com/adobe/aepsdk_flutter/blob/main/plugins/flutter_aepassurance/pubspec.lock.
I suggest that we delete the pubspec.lock files present in each plugin directory and ignore them in .gitignore. The dependency lock file is usually used in the app project, we should not include it in the plugin projects.
cc @cdhoffmann @praveek

remove and gitignore pubspec.lock in the plugins
* update issue template description

update issue template description

* update bug_report.yml

update bug_report.yml
@@ -44,14 +44,14 @@ jobs:

- uses: maxim-lobanov/setup-xcode@v1
with:
xcode-version: "15.0.1"
xcode-version: "15.2"
Copy link
Collaborator

Choose a reason for hiding this comment

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

why are we changing this? We should build on the earliest version we support.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Need to update the versions for all the plugins to 3

Update android project gradle version
migration doc update
@cacheung cacheung merged commit e0d36e8 into staging Apr 23, 2024
10 checks passed
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.

5 participants