Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Applications yaml definition to match IS docs #763

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HasiniSama
Copy link
Contributor

Purpose

$subject

Note:

  • The descriptions/examples are updated to the latest changes done in the docs.
  • The following scopes were aligned to the actual behaviors of the API.

addAuthorizedAPI

  • internal_application_mgt_create - in API
  • internal_application_mgt_update - in YAML
  • internal_application_mgt_update - in docs
  • Correct scope - internal_application_mgt_update

deleteAuthorizedAPI

  • internal_application_mgt_delete - in API
  • internal_application_mgt_update - in YAML
  • internal_application_mgt_update - in docs
  • Correct scope - internal_application_mgt_update

createApplicationTemplate

  • no scope - in API
  • no scope - in YAML
  • internal_application_mgt_create - in docs
  • Correct scope - internal_application_mgt_create

deleteApplicationTemplate

  • no scope - in API
  • no scope - in YAML
  • internal_application_mgt_delete - in docs
  • Correct scope - internal_application_mgt_delete

getAllApplicationTemplates

  • no scope - in API
  • no scope - in YAML
  • internal_application_mgt_view - in docs
  • Correct scope - internal_application_mgt_view

getApplicationTemplate

  • no scope - in API
  • no scope - in YAML
  • internal_application_mgt_view - in docs
  • Correct scope - internal_application_mgt_view

updateApplicationTemplate

  • no scope - in API
  • no scope - in YAML
  • internal_application_mgt_update - in docs
  • Correct scope - internal_application_mgt_update

Goals

To fix the gap between the IS 7.1.0 doc and the API definition.

Related PRs

Related Issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant