Skip to content

Delete dead emancipations.js (#7302) - #7336

Open
KoushikSagarr wants to merge 1 commit into
rubyforgood:mainfrom
KoushikSagarr:7302-remove-emancipations-js
Open

KoushikSagarr wants to merge 1 commit into
rubyforgood:mainfrom
KoushikSagarr:7302-remove-emancipations-js

Conversation

@KoushikSagarr

@KoushikSagarr KoushikSagarr commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

What github issue is this PR for, if any?

Resolves [#7302]

What changed, and why?

  • Deleted the obsolete app/javascript/src/emancipations.js DataTable initialization.
  • Removed the unused require('./src/emancipations') entry from app/javascript/application.js.
  • Confirmed that the emancipation checklist page no longer renders the legacy all-case-emancipations table targeted by this script.
  • Kept changes strictly limited to the obsolete DataTable implementation and its application entry point.

This is another small, independently reviewable contribution toward removing obsolete JavaScript code from CASA.

How is this tested? (please write rspec and jest tests!) 💖💪

  • Jest: 15 test suites passed, 124 tests passed, 0 failures.
  • Asset build: npm run build && npm run build:css passed.
  • RSpec: spec/requests/emancipation_checklists_spec.rb — 4 examples, 0 failures.
  • Reference verification: Confirmed that the legacy DataTable initialization is no longer required by the emancipation checklist page.
  • Git diff: git diff --check passed with no whitespace errors.

Screenshots please :)

Not applicable — this removes an inactive DataTable initializer without introducing UI changes.

Feelings gif (optional)

Another piece of dead code gone. Small changes, cleaner codebase! 💪

@github-actions github-actions Bot added the javascript Touches JavaScript code label Nov 3, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript Touches JavaScript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants