OpenSlides / openslides-manage-service

Manage service for OpenSlides 4+
MIT License
4 stars 12 forks source link

add get sub-command #156

Closed peb-adr closed 2 years ago

peb-adr commented 2 years ago

@normanjaeckel Could you test this too? And you also may have code suggestions (this was my first time coding in go). I'm especially unsure about the data types I'm using in the protocol definition. Maybe you can also uniformize this a bit with the code for the set sub-command

peb-adr commented 2 years ago

@normanjaeckel I tested both get and set now after the rebase. Maybe you could run make protoc on your end again and see if the generated files differ. I got the feeling they will, because we have slightly different setups. So you can either push your proto files on top of this, or just merge at will.

peb-adr commented 2 years ago

@normanjaeckel I tested both get and set now after the rebase. Maybe you could run make protoc on your end again and see if the generated files differ. I got the feeling they will, because we have slightly different setups. So you can either push your proto files on top of this, or just merge at will.

@normanjaeckel could you merge this one or the other way this weekend?