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

Added missing OBO services #667

Merged

Conversation

symphony-soufiane
Copy link
Contributor

@symphony-soufiane symphony-soufiane commented Jun 14, 2022

Description

Based on https://developers.symphony.com/restapi/reference/obo-enabled-endpoints, Some apis that are OBO enabled were missing in OboServices interfaces.
We added them in this PR.
Fixed some warnings as well.

Checklist

  • Referenced an issue in the PR title or description
  • Filled properly the description and dependencies, if any
  • Unit/Integration tests updated or added
  • Javadoc added or updated
  • Updated the documentation in docs folder

Some apis that are OBO enabled were missing in OboServices interfaces.
We added them in this PR.
Fixed some warnings as well.
@symphony-soufiane symphony-soufiane requested a review from a team June 14, 2022 17:12
Copy link
Contributor

@yinan-symphony yinan-symphony left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@symphony-soufiane symphony-soufiane merged commit e213571 into finos:main Jun 15, 2022
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.

2 participants