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

EAM API: add list apps api #298

Merged
merged 2 commits into from
Oct 30, 2024
Merged

EAM API: add list apps api #298

merged 2 commits into from
Oct 30, 2024

Conversation

gainsley
Copy link
Collaborator

What type of PR is this?

  • enhancement/feature

What this PR does / why we need it:

This adds an API call to list created applications. Currently there is no way to query the existing applications outside of knowing the AppIds of created apps, which requires the client side to maintain a database of AppIds. This is not feasible for web or CLI based clients.

Which issue(s) this PR fixes:

Fixes #

Changelog input

 release-note

Additional documentation

docs

Copy link

linux-foundation-easycla bot commented Sep 18, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

Copy link

github-actions bot commented Sep 18, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 2 0 0.02s
✅ JSON eslint-plugin-jsonc 1 0 0 1.33s
✅ JSON jsonlint 1 0 0.18s
✅ JSON prettier 1 1 0 0.98s
✅ JSON v8r 1 0 2.32s
✅ OPENAPI spectral 3 0 6.26s
✅ REPOSITORY git_diff yes no 0.53s
✅ REPOSITORY secretlint yes no 4.52s
✅ YAML yamllint 3 0 0.76s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@gainsley gainsley changed the title Application Management API: add list apps api EAM API: add list apps api Sep 18, 2024
@Kevsy
Copy link
Collaborator

Kevsy commented Oct 15, 2024

Hi @gainsley , just to check, is the intention to list all applications? Or rather the applications owned by the application provider (the API Consumer)?

@gainsley
Copy link
Collaborator Author

Hi @Kevsy, the intent is for all applications, but of course limited by the platform to the applications that the user has permission to see.

@Kevsy
Copy link
Collaborator

Kevsy commented Oct 15, 2024

Thanks @gainsley - please could you add that clarification to the operation description?

@gainsley
Copy link
Collaborator Author

Sure @Kevsy, I've added it, thanks.

@JoseMConde JoseMConde merged commit f9f8268 into main Oct 30, 2024
2 checks passed
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.

3 participants