[Accounts] Fix ACFacebookAudience.OnlyMe mapping in SetPermissions - #26133
Conversation
The switch statement in AccountStoreOptions.SetPermissions mapped ACFacebookAudience.OnlyMe to ACFacebookAudienceValue.Friends, so callers could not request the OnlyMe audience. Map it to ACFacebookAudienceValue.OnlyMe. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Fixes an incorrect mapping in AccountStoreOptions.SetPermissions where ACFacebookAudience.OnlyMe was previously (and silently) mapped to ACFacebookAudienceValue.Friends, preventing callers from requesting the “OnlyMe” audience.
Changes:
- Correct
ACFacebookAudience.OnlyMeto map toACFacebookAudienceValue.OnlyMein the audience switch insideSetPermissions.
This comment has been minimized.
This comment has been minimized.
✅ API diff for current PR / commitNET (empty diffs)✅ API diff vs stableNET (empty diffs)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
🚀 [CI Build #3c1f804] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 203 tests passed 🎉 Tests counts✅ assembly-processing: All 1 tests passed. Html Report (VSDrops) Download macOS tests✅ Tests on macOS Monterey (12): All 5 tests passed. Html Report (VSDrops) Download Linux Build VerificationPipeline on Agent |
Summary
AccountStoreOptions.SetPermissionsmappedACFacebookAudience.OnlyMetoACFacebookAudienceValue.Friendsin its switch statement, so callers could never actually request theOnlyMeaudience — it silently behaved likeFriends.This changes the
ACFacebookAudience.OnlyMecase to useACFacebookAudienceValue.OnlyMe(the field is defined insrc/accounts.csvia[Field ("ACFacebookAudienceOnlyMe")]).The change is minimal and surgical — a single line.
🤖 Pull request created by Copilot