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

Create /graph/operations/all endpoint #2836

Closed
t92549 opened this issue Dec 2, 2022 · 4 comments · Fixed by #3011
Closed

Create /graph/operations/all endpoint #2836

t92549 opened this issue Dec 2, 2022 · 4 comments · Fixed by #3011
Assignees
Labels
feature A proposed new feature good first issue Small, lower complexity and doesn't require pre-existing Gaffer knowledge rest-api Specific to/touches a part of the REST API
Milestone

Comments

@t92549
Copy link
Contributor

t92549 commented Dec 2, 2022

Describe the new feature you'd like
A /graph/operations/all endpoint should be added to return all operations on the classpath.

Why do you want this feature?
The /graph/operations/details/all was added to return the details of all operations on the classpath, but sometimes users don't want details, just the name of the operations.

Additional context
It would be nice if the endpoint returned them sorted alphabetically (see #2775).
For implementation, see the /graph/operations/details/all endpoint.

@t92549 t92549 added enhancement Improvement to existing functionality/feature good first issue Small, lower complexity and doesn't require pre-existing Gaffer knowledge labels Dec 2, 2022
@t92549 t92549 added this to the post-v2.0_backlog milestone Dec 2, 2022
@ibrijesh
Copy link
Contributor

ibrijesh commented Jul 10, 2023

HI , I am interested in contribution, I have already build the project using mvn clean install -Pquick command , I wanted to know how to run these api end point.

@GCHQDeveloper314 GCHQDeveloper314 added rest-api Specific to/touches a part of the REST API feature A proposed new feature and removed enhancement Improvement to existing functionality/feature labels Jul 12, 2023
@GCHQDeveloper314
Copy link
Member

There are Maven profiles in the spring-rest and road-traffic-demo modules. These allow the endpoints (Spring REST or .war with Tomcat) be run locally for testing.
For Spring run mvn compile -Pdemo -pl :spring-rest then go to http://localhost:8080/rest/ and you will see the Swagger site.
For .war run mvn integration-test -Proad-traffic-demo -pl :road-traffic-demo and then go to http://localhost:8080/rest/ and you will see the Swagger site.

The road-traffic-demo also includes some data.

@GCHQDeveloper314 GCHQDeveloper314 linked a pull request Jul 17, 2023 that will close this issue
@GCHQDeveloper314
Copy link
Member

Note that as with #2690 this is only for spring-rest.

GCHQDeveloper314 added a commit that referenced this issue Aug 14, 2023
* New "/graph/operations/all" endpoint for retrieving sorted operation names.

---------

Co-authored-by: GCHQDeveloper314 <[email protected]>
@GCHQDeveloper314 GCHQDeveloper314 linked a pull request Aug 14, 2023 that will close this issue
@GCHQDeveloper314
Copy link
Member

Closed by #3011. However this did not include sorting. Although sorting was part of the original issue above, the potential for sorting can instead be discussed in the issue specific to it (#2775).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A proposed new feature good first issue Small, lower complexity and doesn't require pre-existing Gaffer knowledge rest-api Specific to/touches a part of the REST API
Projects
None yet
3 participants