delphi-hub / delphi-management

The management console for the Delphi platform
https://delphi.cs.uni-paderborn.de/
Apache License 2.0
2 stars 5 forks source link

Show the version of the webapi instance configured #15

Open bhermann opened 6 years ago

bhermann commented 6 years ago

As a authenticated user, I would like to see the version of the webapi instance running for each configured instance.

janniclas commented 5 years ago

@johannesduesing I believe this is desirable for every instance as far as it provides a version, so we might want to extend our instance model to contain a version string? would come in handy to differentiate between possibly incompatible instances too (for example you can only match web apis and web apps with compatible versions).

johannesduesing commented 5 years ago

Sounds like a good idea to me. There are two possible ways of doing this: Either we add a version attribute to the instance model, or Delphi Management just queries the components, since all of them provide an GET /version endpoint. The former solution does require us to change all components again, since the data model changes, but it would allow us to also account for incompatible versions in the matching process. The latter solution is easy to implement, but does not provide any other functionality besides displaying a version string.

@janniclas @bhermann what do you think ?

bhermann commented 5 years ago

I would go with the latter solution. In the matching process you can easily add the call to the endpoint if necessary. For now, I would disregard the problem of incompatible versions.

bhermann commented 4 years ago

@johannesduesing Is this still open?

johannesduesing commented 4 years ago

I think it is, i can't find an implementation in the backend. Do you want me to take this on?