Commit 000ad299 authored by Phil Hughes's avatar Phil Hughes

Merge branch '200009-bug-sentry-integration-shows-project-name-instead-of-slug' into 'master'

Bug: Sentry Integration shows project 'name' instead of 'slug'

Closes #200009

See merge request gitlab-org/gitlab!24176
parents 3c21e5e2 2932d4bb
......@@ -13,6 +13,6 @@ export const transformFrontendSettings = ({ apiHost, enabled, token, selectedPro
return { api_host: apiHost || null, enabled, token: token || null, project };
};
export const getDisplayName = project => `${project.organizationName} | ${project.name}`;
export const getDisplayName = project => `${project.organizationName} | ${project.slug}`;
export default () => {};
---
title: Use project slug instead of name for Error Tracking Settings Display
merge_request: 24176
author:
type: fixed
......@@ -61,7 +61,7 @@ describe 'Projects > Settings > For a forked project', :js do
within('div#project-dropdown') do
click_button('Select project')
click_button('Sentry | Internal')
click_button('Sentry | internal')
end
click_button('Save changes')
......
......@@ -47,7 +47,7 @@ describe('Error Tracking Settings - Getters', () => {
it('should display correctly when a project is selected', () => {
[state.selectedProject] = projectList;
expect(getters.dropdownLabel(state, mockGetters)).toEqual('organizationName | name');
expect(getters.dropdownLabel(state, mockGetters)).toEqual('organizationName | slug');
});
it('should display correctly when no project is selected', () => {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment