lightningrodlabs / rea-playspace

Holochain app to play with REA accounting
32 stars 7 forks source link

Bug: Don't allow operator to pick primary accountable #176

Closed fosterlynn closed 1 year ago

fosterlynn commented 2 years ago

Screenshot from 2022-10-29 13-02-23

The primaryAccountable is set to equal the receiver on the event that creates the resource.

adaburrows commented 2 years ago

It is still set by the event. This was just a general form I created for entering the EconomicResouce values.

*Should we ever allow setting the primary accountable field on an EconomicResource?** Probably not, especially since it just gets overwritten at the next stage of code execution.

We have two choices:

  1. I can remove it from the form entirely.
  2. I can disable it and set it equal to what it will be after the event is applied (so people can see that connection).
fosterlynn commented 2 years ago

We have two choices:

Either one is fine. Might be better to just remove it and keep the form simpler?

adaburrows commented 2 years ago

Aha, we now have a hypothesis to test. Let's keep it and ask people if it provides any value.

fosterlynn commented 2 years ago

Let's keep it and ask people if it provides any value.

Whatever you prefer.

fosterlynn commented 1 year ago

I'm thinking this one should go in the milestone? (We can see what all is there vs what time there is....)