Skip to content

Commit a04090a

Browse files
authored
Remove trailing project status from title (#1057)
to allow user to focus on banner
1 parent fe543f6 commit a04090a

File tree

4 files changed

+1
-12
lines changed

4 files changed

+1
-12
lines changed

app/models/project.rb

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -95,12 +95,7 @@ def metadata=(metadata_model)
9595
end
9696

9797
def title
98-
trailer = if in_mediaflux?
99-
""
100-
else
101-
" (#{::Project::PENDING_STATUS})"
102-
end
103-
self.metadata_model.title + trailer
98+
self.metadata_model.title
10499
end
105100

106101
def departments

spec/system/project_roles_spec.rb

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,6 @@
7171
expect(page).to have_content("Welcome")
7272
click_on("My test project")
7373
expect(page).to have_content("This project has not been saved to Mediaflux")
74-
expect(page).to have_content("My test project (pending)")
75-
expect(page).to have_content("My test project (#{::Project::PENDING_STATUS})")
7674
expect(page).to have_content(read_only.given_name)
7775
expect(page).to have_content(read_only.display_name)
7876
expect(page).to have_content(read_only.family_name)

spec/system/project_show_spec.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -244,7 +244,6 @@
244244
it "shows the sysadmin buttons for a pending project" do
245245
sign_in sysadmin_user
246246
visit "/projects/#{project_not_in_mediaflux.id}"
247-
expect(page).to have_content "#{project_not_in_mediaflux.metadata_model.title} (#{::Project::PENDING_STATUS})"
248247
expect(page).to have_content "This project has not been saved to Mediaflux"
249248
expect(page).to have_content pending_text
250249
expect(page).to have_selector(:link_or_button, "Approve Project")

spec/system/project_spec.rb

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@
4747
it "Shows the not yet approved (pending) project" do
4848
sign_in sponsor_user
4949
visit "/projects/#{project_not_in_mediaflux.id}"
50-
expect(page).to have_content "(#{::Project::PENDING_STATUS})"
5150
expect(page).to have_content pending_text
5251
end
5352
end
@@ -77,7 +76,6 @@
7776
it "shows none when the data user is empty" do
7877
sign_in data_manager
7978
visit "/projects/#{project_not_in_mediaflux.id}"
80-
expect(page).to have_content "project 123 (#{::Project::PENDING_STATUS})"
8179
expect(page).to have_content "This project has not been saved to Mediaflux"
8280
expect(page).to have_content pending_text
8381
expect(page).not_to have_button "Approve Project"
@@ -403,7 +401,6 @@
403401
sign_in sponsor_user
404402
visit "/projects"
405403
expect(page).to have_content(project_not_in_mediaflux.title)
406-
expect(page).to have_content("(#{::Project::PENDING_STATUS})")
407404
end
408405
end
409406

0 commit comments

Comments
 (0)