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

Added sample app with new architecture #424

Merged
merged 31 commits into from
Jan 14, 2025
Merged

Added sample app with new architecture #424

merged 31 commits into from
Jan 14, 2025

Conversation

namArora3112
Copy link
Collaborator

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.

namArora3112 and others added 23 commits January 7, 2025 11:50
Add a sample app with new Arch enabled
* lifecycle initialisation

* lifecycle implementation fixes
* target view integration

* podfile and yarn lock file

* target test mboxes

* build fixes

* Update TargetParameters for platform consistency
* target view integration

* podfile and yarn lock file

* target test mboxes

* Added messaging config

* Added messaging config

* dependecy fixes

* package fixes

* aepsample app new arch CI/CD fixes

* sample app new arch jest fixes

* Remove unsupported --verbose flag in Expo commands

* simulator installation

* ios 17.2

* removed simulator

* simulator path set

* xcode build fixes

* removed files
* Add documentation for Expo Support (#377)

* update docs for Expo support

* update expo.md

* update README

* Add screenshots for docs

* Update docs/expo.md

* Apply suggestions from code review

* address PR comments

* address PR comments

* address PR comments

* address PR comments

* update code snippets
Adobe Mobile SDK setup workaround with Expo from @JoeM-RP, created AdobeBridge.h and AdobeBridge.m to wrap the SDK setup and imported it into AppDelegate.mm.  Thanks for the great solution!

* add info for lifecycle docs

---------

Co-authored-by: Mayank Shekhar <[email protected]>

* Update development doc

Update development doc

* podfile and yarn lock file

* build fixes

* messaging view integration

* messagong view navigation fixes

* Revert "Add documentation for Expo Support (#377)"

This reverts commit 28f814a.

* Revert "podfile and yarn lock file"

This reverts commit 5f52ede.

* Revert "build fixes"

This reverts commit 403cd10.

* router fixes and messagive view added to home screen

---------

Co-authored-by: Ishita Gambhir <[email protected]>
Co-authored-by: Mayank Shekhar <[email protected]>
Co-authored-by: Calise Cheung <[email protected]>
* target view integration

* podfile and yarn lock file

* Added messaging config

* Added messaging config

* package fixes

* aepsample app new arch CI/CD fixes

* simulator installation

* ios 17.2

* removed simulator

* removed files

* places sdk fies

* places native module name fixes in android

* places sdk

* places config

* removed unused code
* Read me updated for AEPSampleNewArcEnabled

* read me fixes

* Update README.md

* Update README.md

* android app id placeholder consistent with iOS along with read me update
React native new architecture sample app
Added places target navigation on main home page
@namArora3112 namArora3112 changed the title Staging Added sample app with new architecture Jan 10, 2025
cacheung and others added 4 commits January 13, 2025 17:34
Update AEPSampleApp Readme
update readme with yarn install
Align extension view names with older arc sample app for consistency
@namArora3112 namArora3112 merged commit a35b315 into main Jan 14, 2025
5 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.

3 participants