Skip to content

WIP - Libscholar 46 update home page#1190

Open
Janell-Huyck wants to merge 12 commits intodevelopfrom
LIBSCHOLAR-46-update-home-page
Open

WIP - Libscholar 46 update home page#1190
Janell-Huyck wants to merge 12 commits intodevelopfrom
LIBSCHOLAR-46-update-home-page

Conversation

@Janell-Huyck
Copy link
Contributor

@Janell-Huyck Janell-Huyck commented Dec 1, 2025

Waiting for #1189 to be merged first and to do a rebase once that's done


Update Home Page and Help Page Links

This PR updates the home page by removing external links and reorganizing them, and updates the IT@UC partner link and icon.

Changes

Home Page Updates

  • Removed external links section: Removed "Scholar Blog" and "Digital Scholarship" links from the home page footer
  • Removed empty links partial: Deleted app/views/layouts/scholar/_links.html.erb and its render call since all links were removed
  • Moved "Manage Your Data" link: Relocated the "Manage Your Data" link from the home page to the Help page, where it appears in the help resources list with a description

Partner Section Updates

  • Updated IT@UC link: Changed the IT@UC partner link URL from http://ucit.uc.edu to https://www.uc.edu/about/ucit.html to use the updated Digital Technology Solutions page
  • Updated IT@UC icon: Replaced the ituc.png logo with dts_uc.png to reflect the rebranding from IT@UC to Digital Technology Solutions (DTS)

Test Updates

  • Removed outdated homepage test: Removed the "shows external links" test from spec/features/hyrax/homepage_spec.rb since the external links section no longer exists
  • Added help page test: Added a test in spec/views/static/help.html.erb_spec.rb to verify the "Manage Your Data" link appears on the help page
  • Disabled flaky test: Disabled the flaky collection delete feature spec in CI to prevent intermittent test failures

Files Changed

  • app/views/hyrax/homepage/index.html.erb - Removed render call for links partial
  • app/views/layouts/scholar/_links.html.erb - Deleted (no longer needed)
  • app/views/layouts/scholar/_partner_branding.html.erb - Updated IT@UC URL and icon to DTS
  • app/views/static/help.html.erb - Added "Manage Your Data" link
  • spec/features/hyrax/homepage_spec.rb - Removed external links test
  • spec/views/static/help.html.erb_spec.rb - Added Manage Your Data link test
  • spec/features/hyrax/dashboard/collection_spec.rb - Disabled flaky test

…ization

- Remove SimpleCov/Coveralls setup from spec_helper.rb to prevent double initialization
- Consolidate all coverage setup in rails_helper.rb
- Add simplecov-lcov and configure LCOV formatter for CircleCI
- Use SimpleCov.formatters (plural) with all three formatters: HTML, LCOV, and Coveralls
- Configure formatters before starting SimpleCov to ensure proper setup
- Replace SimpleCov.start with Coveralls.wear! to properly handle parallel test execution
- Configure formatters before Coveralls.wear! starts SimpleCov
- Maintains LCOV formatter for CircleCI upload
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant