Skip to content

Conversation

@mavwolverine
Copy link
Contributor

@mavwolverine mavwolverine commented Sep 2, 2024

The instructions specify to enable Developer Mode in Privacy and Security. But Developer Mode option is not available, unless you open Xcode and reconnect the device.


9/23 edit from @andrewkolos: See https://stackoverflow.com/questions/77648341/flutter-not-reading-device-developer-mode-setting

Pre-launch Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@flutter-dashboard
Copy link

It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption, contact "@test-exemption-reviewer" in the #hackers channel in Discord (don't just cc them here, they won't see it!).

If you are not sure if you need tests, consider this rule of thumb: the purpose of a test is to make sure someone doesn't accidentally revert the fix. Ask yourself, is there anything in your PR that you feel it is important we not accidentally revert back to how it was before your fix?

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. The test exemption team is a small volunteer group, so all reviewers should feel empowered to ask for tests, without delegating that responsibility entirely to the test exemption group.

@google-cla
Copy link

google-cla bot commented Sep 2, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@github-actions github-actions bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Sep 2, 2024
@andrewkolos
Copy link
Contributor

Converting to draft since some tests need to be updated (see the ones in failing tools_tests_general checks).

@andrewkolos andrewkolos marked this pull request as draft September 10, 2024 19:30
@flutter-dashboard
Copy link

This pull request has been changed to a draft. The currently pending flutter-gold status will not be able to resolve until a new commit is pushed or the change is marked ready for review again.

For more guidance, visit Writing a golden file test for package:flutter.

Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing.

@mavwolverine mavwolverine force-pushed the patch-1 branch 3 times, most recently from c523627 to a3f4aad Compare September 10, 2024 20:39
@mavwolverine mavwolverine marked this pull request as ready for review September 11, 2024 17:28
@andrewkolos
Copy link
Contributor

andrewkolos commented Sep 12, 2024

The instructions specify to enable Developer Mode in Privacy and Security. But Developer Mode option is not available, unless you open Xcode and reconnect the device.

@jmagman Have you ever heard of this before? That is, having to open Xcode and disconnect/reconnect your device before being able to enable Developer Mode on the device? I'm questioning whether or not there is actually a tool bug here.

@mavwolverine
Copy link
Contributor Author

mavwolverine commented Sep 12, 2024

The instructions specify to enable Developer Mode in Privacy and Security. But Developer Mode option is not available, unless you open Xcode and reconnect the device.

@jmagman Have you ever heard of this before? That is, having to open Xcode and disconnect/reconnect your device before being able to enable Developer Mode on the device? I'm questioning whether or not there is actually a tool bug here.

That is what worked for me, so I added the PR. Also see
https://stackoverflow.com/questions/77648341/flutter-not-reading-device-developer-mode-setting

Comment on lines 31 to 32
String flutterSpecifiedDeviceDevModeDisabled(String deviceName) => 'To use '
"'$deviceName' for development, enable Developer Mode in Settings → Privacy & Security.";
"'$deviceName' for development, open Xcode, reconnect the '$deviceName' and enable Developer Mode in Settings → Privacy & Security.";
Copy link
Contributor

Choose a reason for hiding this comment

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

Also see
https://stackoverflow.com/questions/77648341/flutter-not-reading-device-developer-mode-setting

Thanks for the link! I can see now that this is a legitimate UX problem.

I think the text change proposed here is a little confusing (though it's possible this is due to me not testing on physical iOS devices very often). When I first read it, I thought it was telling me to change settings in Xcode, not on the device.

Referencing the StackOverflow thread linked, I propose this as a starting point:

To use <device> for development, enable Developer Mode in Settings → Privacy & Security on your device.

If this does not work, open Xcode, reconnect the device, and look for a popup on the device asking you to trust this computer.

Copy link
Member

Choose a reason for hiding this comment

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

@andrewkolos I agree, your wording is better.

Copy link
Contributor

Choose a reason for hiding this comment

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

Updated in commit reword message

@andrewkolos andrewkolos self-requested a review October 8, 2024 20:28
@bkonyi bkonyi self-requested a review October 10, 2024 20:57
@andrewkolos andrewkolos removed their request for review October 11, 2024 20:06
@andrewkolos
Copy link
Contributor

Unlisting myself as a reviewer since I have directly pushed changes (context: #154511 (comment))

@jmagman
Copy link
Member

jmagman commented Oct 12, 2024

Unlisting myself as a reviewer since I have directly pushed changes (context: #154511 (comment))

@andrewkolos I hereby bless you to give you the second LGTM 🙏

@virajkanwade Thank you for the contribution!

engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 12, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 12, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 13, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 13, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 14, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 16, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Oct 17, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 12, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 13, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 6, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmit Merge PR when tree becomes green via auto submit App tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants