Skip to content

Conversation

@ewpatton
Copy link
Member

@ewpatton ewpatton commented Nov 8, 2025

General items:

If your code changes how something works on the device (i.e., it affects the companion):

  • I branched from ucr
  • My pull request has ucr as the base

Further, if you've changed the blocks language or another user-facing designer/blocks API (added a SimpleProperty, etc.):

  • I have updated the corresponding version number in appinventor/components/src/.../common/YaVersion.java
  • I have updated the corresponding upgrader in appinventor/appengine/src/.../client/youngandroid/YoungAndroidFormUpgrader.java (components only)
  • I have updated the corresponding entries in appinventor/blocklyeditor/src/versioning.js

What does this PR accomplish?

As discussed at the dev planning meeting, the default value of the PlayOnlyInForeground property made our existing static analysis for whether to apply permissions not work w.r.t. iOS. This PR changes the default value so that the lack of this property in the code guarantees that an app built for iOS can rely on its presence to set the audio background mode property. The presence of this flag is scrutinized and the current system causes people using the Player in published apps to have to justify the audio background mode's presence.

Change-Id: I1495dc33d894db25be9439b34a4f167dfee64e1b
Change-Id: Id07c5b88e2e41ceb078338471ab5ca59041814f8
@ewpatton ewpatton force-pushed the feature/swap-player-defaults branch from 6b227c2 to d9764a9 Compare November 8, 2025 13:58
@ewpatton ewpatton requested a review from a team November 10, 2025 16:34
@ewpatton ewpatton added this to the nb203 milestone Nov 10, 2025
@jisqyv jisqyv added the on ai2-test Code is deployed on ai2-test.appinventor.mit.edu label Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants