Skip to content

Conversation

@jkasten2
Copy link
Member

@jkasten2 jkasten2 commented Aug 4, 2021

  • Bumped dev app version to 1.4.4
  • fix dev app archiving with multiple embedding
    • OneSignal framework was being embedded more than once due to the NSE and AppClip also emending it.

This change is Reviewable

* OneSignal framework was being embeded more than once due to
the NSE and AppClip also emdeding it.
@jkasten2 jkasten2 changed the title Dev app/bump to 1.4.4 and fix target embedding Dev app - Bump to 1.4.4 and fix target embedding Aug 5, 2021
Copy link
Contributor

@adamschlesinger adamschlesinger left a comment

Choose a reason for hiding this comment

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

This PR looks similar to what I had thrown together in #959. IIRC the dev builds were fixed but on submission we still experienced

ITMS-90562: Invalid Bundle - One or more dynamic libraries that are referenced by your app are not present in the dylib search path.

Reviewable status: 0 of 1 files reviewed, all discussions resolved (waiting on @emawby and @Jeasmine)

Copy link
Member Author

@jkasten2 jkasten2 left a comment

Choose a reason for hiding this comment

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

I was getting that as well, the changes in this PR got me through the whole process.

Reviewable status: 0 of 1 files reviewed, all discussions resolved (waiting on @Jeasmine)

@jkasten2 jkasten2 merged commit 7644e86 into main Aug 27, 2021
@jkasten2 jkasten2 deleted the dev_app/bump_to_1.4.4_and_fix_target_embeding branch August 27, 2021 22:55
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.

4 participants