Skip to content

Remove legacy case study code [WHIT-3250]#11451

Merged
kashifatcha merged 1 commit into
mainfrom
remove-legacy-case-study-code-v2
May 20, 2026
Merged

Remove legacy case study code [WHIT-3250]#11451
kashifatcha merged 1 commit into
mainfrom
remove-legacy-case-study-code-v2

Conversation

@kashifatcha
Copy link
Copy Markdown
Contributor

@kashifatcha kashifatcha commented May 14, 2026

This commit removes legacy case study MVC:

  • model
  • views
  • controllers
  • factories
  • presenters

It also removes lead image component related code.

@kashifatcha kashifatcha force-pushed the remove-legacy-case-study-code-v2 branch 6 times, most recently from 7531dae to 5faea0c Compare May 19, 2026 13:00
@kashifatcha kashifatcha changed the title Remove legacy case study code Remove legacy case study code [WHIT-3250] May 19, 2026
@kashifatcha kashifatcha marked this pull request as ready for review May 19, 2026 13:38
Copy link
Copy Markdown
Contributor

@ChrisBAshton ChrisBAshton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, Kash! Just a couple more things to delete (feel free to fix it up to the same commit), then we're good to go.

Comment thread config/routes.rb Outdated
Comment thread test/components/admin/edition_images/image_component_test.rb Outdated
class ShareablePreviewIntegrationTest < ActionDispatch::IntegrationTest
extend Minitest::Spec::DSL
include TaxonomyHelper
include Admin::EditionRoutesHelper
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

Comment thread test/unit/lib/whitehall/publishing_api_test.rb
This commit removes legacy case study MVC:

- model
- views
- controllers
- factories
- presenters

It also removes lead image component related code.
@kashifatcha kashifatcha force-pushed the remove-legacy-case-study-code-v2 branch from 5faea0c to 1aaff63 Compare May 19, 2026 14:14
@kashifatcha kashifatcha merged commit 0f5896d into main May 20, 2026
25 checks passed
@kashifatcha kashifatcha deleted the remove-legacy-case-study-code-v2 branch May 20, 2026 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants