I think it's fine, even though it's a little weird to send OK to ignore an
error instead of Ignore.

Rob said:
    I think you want a call to UserExperienceCheckExecuteResult() here to
get nResult interpreted correctly. That will remove the question in you
switch statement below.

Since you're handling OK and NoAction differently, I don't think you can
use UserExperienceCheckResult but you should be able to coerce the nResult
in a similar manner.

On Fri, Oct 31, 2014 at 10:50 AM, Hoover, Jacob <jacob.hoo...@greenheck.com>
wrote:

>  Finally got around to making that small breaking tweak to the interface
> to allow the detect process to continue on after a failed detect update if
> the BA chooses to do so.
>
>
>
> https://github.com/jchoover/wix4/tree/DetectUpdateComplete
>
>
>
> I haven’t done a pull request yet, as I need to test/verify my changes.
> However if anyone wants to critiique the implementation I’m all ears.
>
>
>
> Thanks,
>
> Jacob
>
>
>
>
> ------------------------------------------------------------------------------
>
> _______________________________________________
> WiX-devs mailing list
> WiX-devs@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/wix-devs
>
>
------------------------------------------------------------------------------
Comprehensive Server Monitoring with Site24x7.
Monitor 10 servers for $9/Month.
Get alerted through email, SMS, voice calls or mobile push notifications.
Take corrective actions from your mobile device.
http://pubads.g.doubleclick.net/gampad/clk?id=154624111&iu=/4140/ostg.clktrk
_______________________________________________
WiX-devs mailing list
WiX-devs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wix-devs

Reply via email to