Resolve "Update gRPC abstraction API with missing calls and refactoring"
All threads resolved!
Description
Complete:
-
User -
Role -
Pnd - To dicuss: CreatePnd call, uses CreatePndList, should name of the API call be the same even if we only gice the option to add one? Can we make it optional to add multiple in the API call too?
- Maybe do only one call where a list of one or many is provided?
-
Plugin
- Maybe move GetPluginSchemaTree from MNE to Plugin?
-
MNE
- For creation: Maybe do only one call where a list of one or many is provided?
- Need to discuss Update function again!
-
Auth
Missing for now. Maybe implement?:
-
ConfigurationManagement -
Route (Not implemented because this will probably get reworked) -
Topology (Not implemented because this will probably get reworked) -
App
Related to #359
Edited by Fabian Seidl
Merge request reports
Activity
added componentnbi goSDN priorityhigh statusavailable typeimprovement labels
added 7 commits
- 206f3a9f - rename pnd requests in API to fit definition
- 8501dc52 - add getAllMne and getAllPnd
- f57e7e7a - add mne update to API, needs some considerations after disccussion
- 7a87d5c5 - add API stuff for configuration management
- 918a10c6 - add API stuff for App service
- d6a3d761 - remove unused param
- c1b6cb38 - fix missing error handling
Toggle commit list- Resolved by Fabian Seidl
Do we want to add tests for the newly added abstractions?
- Resolved by Fabian Seidl
Make sure all questions in description are answered
added 12 commits
-
c1b6cb38...1746a763 - 5 commits from branch
master
- 5922b50e - rename pnd requests in API to fit definition
- 65630a13 - add getAllMne and getAllPnd
- e17e6adc - add mne update to API, needs some considerations after disccussion
- 9c33424a - add API stuff for configuration management
- 167c4965 - add API stuff for App service
- 7b3bceed - remove unused param
- c9fe9503 - fix missing error handling
Toggle commit list-
c1b6cb38...1746a763 - 5 commits from branch
added 8 commits
- 206f3a9f - rename pnd requests in API to fit definition
- 8501dc52 - add getAllMne and getAllPnd
- f57e7e7a - add mne update to API, needs some considerations after disccussion
- 7a87d5c5 - add API stuff for configuration management
- 918a10c6 - add API stuff for App service
- d6a3d761 - remove unused param
- c1b6cb38 - fix missing error handling
- 5317ec79 - Merge branch '359-update-grpc-abstraction-api-with-missing-calls' of...
Toggle commit listadded 7 commits
-
5317ec79...e90d85af - 6 commits from branch
master
- bbc8372f - Merge branch 'master' into 359-update-grpc-abstraction-api-with-missing-calls
-
5317ec79...e90d85af - 6 commits from branch
added 5 commits
-
bbc8372f...d149b5ae - 4 commits from branch
master
- 5fb42105 - Merge branch 'master' into 359-update-grpc-abstraction-api-with-missing-calls
-
bbc8372f...d149b5ae - 4 commits from branch
added 4 commits
-
5fb42105...d760fd94 - 3 commits from branch
master
- 8ce3750e - Merge branch 'master' into 359-update-grpc-abstraction-api-with-missing-calls
-
5fb42105...d760fd94 - 3 commits from branch
added 4 commits
-
8ce3750e...0515e7b1 - 3 commits from branch
master
- eedc738c - Merge branch 'master' into 359-update-grpc-abstraction-api-with-missing-calls
-
8ce3750e...0515e7b1 - 3 commits from branch
added 10 commits
-
eedc738c...c026ea5b - 8 commits from branch
master
- df96d040 - Merge branch 'master' into 359-update-grpc-abstraction-api-with-missing-calls
- 69a0641d - move pluginshema thingy and remove comment
-
eedc738c...c026ea5b - 8 commits from branch
requested review from @hda11597, @neil.schark, and @istkarenk
mentioned in commit 66ac272b
added statuscompleted label and removed statusavailable label
Please register or sign in to reply