Skip to content

Commit

Permalink
Remove trailing project status from title (#1057)
Browse files Browse the repository at this point in the history
to allow user to focus on banner
  • Loading branch information
leefaisonr authored Nov 18, 2024
1 parent fe543f6 commit a04090a
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 12 deletions.
7 changes: 1 addition & 6 deletions app/models/project.rb
Original file line number Diff line number Diff line change
Expand Up @@ -95,12 +95,7 @@ def metadata=(metadata_model)
end

def title
trailer = if in_mediaflux?
""
else
" (#{::Project::PENDING_STATUS})"
end
self.metadata_model.title + trailer
self.metadata_model.title
end

def departments
Expand Down
2 changes: 0 additions & 2 deletions spec/system/project_roles_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,6 @@
expect(page).to have_content("Welcome")
click_on("My test project")
expect(page).to have_content("This project has not been saved to Mediaflux")
expect(page).to have_content("My test project (pending)")
expect(page).to have_content("My test project (#{::Project::PENDING_STATUS})")
expect(page).to have_content(read_only.given_name)
expect(page).to have_content(read_only.display_name)
expect(page).to have_content(read_only.family_name)
Expand Down
1 change: 0 additions & 1 deletion spec/system/project_show_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,6 @@
it "shows the sysadmin buttons for a pending project" do
sign_in sysadmin_user
visit "/projects/#{project_not_in_mediaflux.id}"
expect(page).to have_content "#{project_not_in_mediaflux.metadata_model.title} (#{::Project::PENDING_STATUS})"
expect(page).to have_content "This project has not been saved to Mediaflux"
expect(page).to have_content pending_text
expect(page).to have_selector(:link_or_button, "Approve Project")
Expand Down
3 changes: 0 additions & 3 deletions spec/system/project_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@
it "Shows the not yet approved (pending) project" do
sign_in sponsor_user
visit "/projects/#{project_not_in_mediaflux.id}"
expect(page).to have_content "(#{::Project::PENDING_STATUS})"
expect(page).to have_content pending_text
end
end
Expand Down Expand Up @@ -77,7 +76,6 @@
it "shows none when the data user is empty" do
sign_in data_manager
visit "/projects/#{project_not_in_mediaflux.id}"
expect(page).to have_content "project 123 (#{::Project::PENDING_STATUS})"
expect(page).to have_content "This project has not been saved to Mediaflux"
expect(page).to have_content pending_text
expect(page).not_to have_button "Approve Project"
Expand Down Expand Up @@ -403,7 +401,6 @@
sign_in sponsor_user
visit "/projects"
expect(page).to have_content(project_not_in_mediaflux.title)
expect(page).to have_content("(#{::Project::PENDING_STATUS})")
end
end

Expand Down

0 comments on commit a04090a

Please sign in to comment.