Replies: 2 comments 1 reply
-
|
I sort of agree. I think we should instead expose a client version so that we can do something like
I think this is generally how we should do our APIs e.g. do the same for BlueAPI etc. I don't mean to say people can't just use the REST calls if they like, we should still let them but sometimes its nicer to have a prebuilt client. |
Beta Was this translation helpful? Give feedback.
-
|
Right now we have the dependency in dodal so that dodal can use the client class and use the model classes. The config class could be defined anywhere really, we just thought it should go in this repo so that things other than dodal could use it. As for the model classes, as it is right now, the server itself uses these to tell it how to convert things. Dodal also needs to use them so it can request that the server correctly converts the files.
This sounds good. That can install both bits I just mentioned |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Currently daq-config-server is a package dependency in dodal, howerer I think initially the daq-config-server was more of a "software as a service" (correct me if I am wrong). Ideally all we need to get a lookuptable from server is to make a get request to an endpoint of a server, so we do not actually need the whole package to be imported to dodal but instead just an http request (can be a sort of light client as well). Does it make sense?
Beta Was this translation helpful? Give feedback.
All reactions