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

GET_REPORTS API #138

Merged
merged 2 commits into from
May 1, 2024
Merged

GET_REPORTS API #138

merged 2 commits into from
May 1, 2024

Conversation

yousiefc
Copy link

PR Description

Added API to generate reports

Description

Related Issue

Motivation and Context

How Has This Been Tested?

API tester client

Screenshots (if appropriate):

@yousiefc yousiefc self-assigned this Apr 30, 2024
@yousiefc yousiefc added the API Awareness for the API Team label Apr 30, 2024
Copy link
Contributor

@akshaypuli akshaypuli left a comment

Choose a reason for hiding this comment

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

Thank You for the PR! Looks great! Just a few comments. Could you also upload the YAML test file?

src/api/api/BIOMD/BIOMD.js Outdated Show resolved Hide resolved
src/api/api/BIOMD/BIOMD.js Outdated Show resolved Hide resolved
src/api/api/BIOMD/BIOMD.js Outdated Show resolved Hide resolved
src/api/services/Communication_Link/middleware.js Outdated Show resolved Hide resolved
src/api/api/BIOMD/BIOMD.js Outdated Show resolved Hide resolved
oaedward
oaedward previously approved these changes May 1, 2024
@yousiefc yousiefc changed the title New API that returns reports GET_REPORTS API May 1, 2024
@akshaypuli akshaypuli merged commit cc00bd4 into develop May 1, 2024
@akshaypuli akshaypuli deleted the feature/report-api branch May 1, 2024 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Awareness for the API Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants