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.

Sign up for GitHub

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

Jump to bottom

Move elastic profiles /usages api to internal path #5583

Merged

bdpiprava merged 1 commit into gocd:master from vishaldevgire:move_elastic_profile_usage_api_to_intenal

Dec 19, 2018

Contributor

vishaldevgire commented Dec 17, 2018 •

edited

Changes:

Created a new controller for api/internal/elastic/profiles/profile_id/usages API (this is required since the base path has changed)
Moved ElasticProfileUsageRepresenter class to new package
Changed ElasticProfileSPA to use internal path for /usages API


          Move elastic profiles /usages api to internal path

90781ea

vishaldevgire requested a review from bdpiprava

December 17, 2018 08:50

Contributor

GaneshSPatil commented Dec 17, 2018

This is not necessary for the current release. Marking it for 19.1.0

GaneshSPatil added this to the Release 19.1.0 milestone

bdpiprava merged commit 439e3a5 into gocd:master

bdpiprava deleted the move_elastic_profile_usage_api_to_intenal branch

December 19, 2018 14:12

maheshp added this to In progress in 19.1.0 via automation

maheshp moved this from In progress to Done in 19.1.0

rajiesh moved this from Done to QA Done in 19.1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment