Page MenuHomePhabricator
Differential D7542

[native] Fix error check condition
ClosedPublic
Actions

Authored by atul on Apr 19 2023, 1:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 1, 9:13 AM
Unknown Object (File)
Fri, Nov 1, 9:13 AM
Unknown Object (File)
Fri, Nov 1, 9:13 AM
Unknown Object (File)
Fri, Nov 1, 9:12 AM
Unknown Object (File)
Fri, Nov 1, 9:10 AM
Unknown Object (File)
Oct 1 2024, 12:31 PM
Unknown Object (File)
Oct 1 2024, 12:31 PM
Unknown Object (File)
Oct 1 2024, 12:31 PM
View All 81 Files
Subscribers
tomek

Details

Reviewers
ashoat
ginsu
Commits
rCOMM4fdefafb4868: [native] Fix error check condition
Summary

In the D7530 Test Plan I mention:

Flipped boolean and observed that Alert appeared as expected

Well, looks like I included the flipped condition in the diff... thankfully it was easy to catch.

Test Plan

Setting image avatar works again

Diff Detail

Repository
rCOMM Comm
Branch
arcpatch-D7542 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
native/
avatars/
2 lines

Diff 25411

View Options

native/avatars/avatar-hooks.js

Loading...