AbsaOSS / AbsaOSS/enceladus

Menas API: replace arrays with objects as top-level JSON reponses

オープン
#1,616 コメント 0 件 リアクション 1 件 担当者 0 名 GitHub で見る
feature priority: undecided under discussion
主要言語
Scala
スター
33
フォーク
16
PR マージ指標
30日以内にマージされた PR はありません

説明

## Background
As outlined in https://github.com/AbsaOSS/enceladus/pull/1597#discussion_r540049514, Menas API contains top-level arrays in response payloads in a number of endpoints. For reasons outline in the lined discussion, these should be replaced by wrapper objects.

API calls in question:
- `/api/properties/datasets`
- `/api/runs/{summaries, {datasetName}, grouped, /grouped/{datasetName}, /{datasetName}/{datasetVersion}}`
- `/{list, searchSuggestions, /allVersions/{name}}`

## Feature
There should be an (extensible) object as a response in JSON.

## Example
`GET /api/properties/datasets` now yields:
```json
[
{
"name": "datasetProperty1",
...
},
{
"name": "datasetProperty2",
...
}
]
```
The intended response should be e.g.:
```json
{
"propertyDefinitions" : [
{
"name": "datasetProperty1",
...
},
{
"name": "datasetProperty2",
...
}
]
}
```

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。