isogr / register-system-transition

Covers GR system transition from 2013 Java-based version to 2023 static site based version
1 stars 1 forks source link

sql2yaml: we should migrate management information #45

Closed strogonoff closed 6 months ago

strogonoff commented 7 months ago

In the management information of transformations 484 or 739, the following data is missing that we should migrate:

stefanomunarini commented 7 months ago

How should this data be modelled (AKA what data format is Paneron expecting)? @strogonoff

strogonoff commented 7 months ago

You can create a proposal & see YAML for yourself.

I created one as an example, see registerManagerNotes, controlBodyNotes & dates: https://github.com/isogr/registry/blob/master/gr-registry/proposals/06c61279-3f84-4b1a-99d0-05c6cfb7cff1/main.yaml

strogonoff commented 7 months ago

To describe the sponsor, the “stakeholder” field on the “propose” transition can be used.

stefanomunarini commented 7 months ago

@strogonoff I have updated the data to include "register_manager_notes". I have changed timeproposed and timedisposed to dateproposed and datedisposed. I have added the name of the sponsor but data is modelled differently than what you expect (based on your comment just above). We use, in fact, the sponsor field to include this piece of data.

Can you have a look at one proposal included in this PR and let me know if this is what you are expecting? Thanks

stefanomunarini commented 7 months ago

You can create a proposal & see YAML for yourself.

Can we have some documentations on how to perform operations in Paneron? I honestly cannot remember how to create a Proposal (see also https://github.com/isogr/register-system-transition/issues/44#issuecomment-1954134904), as the system does not allow me to perform such operation

For some reason now I am able to. Can this be something related to caching or some settings that need to be reset in the Paneron app? I simply restarted the app and the button was clickable.

strogonoff commented 7 months ago

Perhaps, would be good to reproduce later…

strogonoff commented 7 months ago

I see register_manager_notes, that’s incorrect spelling, should be registerManagerNotes. For timestamps, instead of dateproposed and datedisposed we use timeProposed and timeDisposed.

stefanomunarini commented 7 months ago

Missing data has been included in https://github.com/isogr/registry/commit/0d8876a53d46247bcc6d3f766a1738e5f78a33f2

stefanomunarini commented 7 months ago

Can we close this @strogonoff ?