diff options
author | Tim Zallmann <tzallmann@gitlab.com> | 2018-10-11 15:18:44 +0000 |
---|---|---|
committer | Tim Zallmann <tzallmann@gitlab.com> | 2018-10-11 15:18:44 +0000 |
commit | a8513c7dc306aee838074bdd01c1999d8c19f4c3 (patch) | |
tree | b2d01799b91dd1dfb6041ce71fec35a8f0fbfac8 | |
parent | 2343dd8dd6ef6c74511e34808377b7dc5fe8c25b (diff) | |
parent | 8bc33c789443550ce2f0d75cf8477e1aae204538 (diff) | |
download | gitlab-ce-a8513c7dc306aee838074bdd01c1999d8c19f4c3.tar.gz |
Merge branch '52472-pipeline-endpoint-json' into 'master'
Adds `.json` to the end of pipelines endpoint
Closes #52472
See merge request gitlab-org/gitlab-ce!22293
-rw-r--r-- | app/assets/javascripts/jobs/store/actions.js | 2 | ||||
-rw-r--r-- | changelogs/unreleased/52472-pipeline-endpoint-json.yml | 5 | ||||
-rw-r--r-- | spec/javascripts/jobs/store/actions_spec.js | 4 |
3 files changed, 8 insertions, 3 deletions
diff --git a/app/assets/javascripts/jobs/store/actions.js b/app/assets/javascripts/jobs/store/actions.js index 298367c9342..c8e1090bcc5 100644 --- a/app/assets/javascripts/jobs/store/actions.js +++ b/app/assets/javascripts/jobs/store/actions.js @@ -139,7 +139,7 @@ export const fetchStages = ({ state, dispatch }) => { dispatch('requestStages'); axios - .get(state.job.pipeline.path) + .get(`${state.job.pipeline.path}.json`) .then(({ data }) => { dispatch('receiveStagesSuccess', data.details.stages); dispatch('fetchJobsForStage', data.details.stages[0]); diff --git a/changelogs/unreleased/52472-pipeline-endpoint-json.yml b/changelogs/unreleased/52472-pipeline-endpoint-json.yml new file mode 100644 index 00000000000..feff195beb8 --- /dev/null +++ b/changelogs/unreleased/52472-pipeline-endpoint-json.yml @@ -0,0 +1,5 @@ +--- +title: Fix caching issue with pipelines URL +merge_request: 22293 +author: +type: fixed diff --git a/spec/javascripts/jobs/store/actions_spec.js b/spec/javascripts/jobs/store/actions_spec.js index 5ab1f75d0d6..ce07effba9e 100644 --- a/spec/javascripts/jobs/store/actions_spec.js +++ b/spec/javascripts/jobs/store/actions_spec.js @@ -422,7 +422,7 @@ describe('Job State actions', () => { beforeEach(() => { mockedState.job.pipeline = { - path: `${TEST_HOST}/endpoint.json/stages`, + path: `${TEST_HOST}/endpoint`, }; mock = new MockAdapter(axios); }); @@ -434,7 +434,7 @@ describe('Job State actions', () => { describe('success', () => { it('dispatches requestStages and receiveStagesSuccess, fetchJobsForStage ', done => { mock - .onGet(`${TEST_HOST}/endpoint.json/stages`) + .onGet(`${TEST_HOST}/endpoint.json`) .replyOnce(200, { details: { stages: [{ id: 121212, name: 'build' }] } }); testAction( |