normative(PaymentComplete): replace '' with 'unknown' (closes #308)

[marcoscaceres]

No description provided.

[marcoscaceres]

Hmm... maybe this should be "abort". @rsolomakhin, do you have any thoughts on this?

[rsolomakhin]

I would prefer not to change this part of the spec. If we do change it, complete('unknown') matches the meaning of complete(''). Let's not use 'abort', because it can be confused with abort() method, whose meaning and behavior are different.

[adrianba]

It's definitely not 'abort' - that would be a very different meaning. You call this with '' or 'unknown' when the transaction is in progress not when you are aborting.

[marcoscaceres]

@rsolomakhin, yeah, ignore my "abort" suggestion.

However, I'd really like to see "" changed to "unknown" on the Moz side for the reasons I mentioned in #308.

[zkoch]

This LGTM