Skip to content

Remove unused GRID, LaunchDarkly, and legacy command code - #575

Merged
jrhoads merged 2 commits into
devfrom
quickwin/delete-dead-code
Sep 28, 2026
Merged

jrhoads merged 2 commits into
devfrom
quickwin/delete-dead-code

Conversation

@jrhoads

@jrhoads jrhoads commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Purpose

Remove unused GRID conversion commands, the LaunchDarkly call site, and other dead code that is no longer part of the running API.

Approach

Delete the legacy* management commands and the settings that only those commands used. Drop the empty GRID_REMOVED_IDS check in organization retrieve. Remove features.py and the redundant v2 client test module, and drop the duplicate CountryBucket class. Leave middleware and the single CommonMiddleware entry in place. Bring the branch up to date with dev by keeping the legacy command deleted rather than restoring a logging tweak inside it, and correct ARCHITECTURE.md so it no longer describes that removed code.

Key Modifications

  • Delete rorapi/management/commands/legacy*.py.
  • Remove the GRID / LEGACY_ROR_DUMP settings block, GRID_REMOVED_IDS, and LAUNCH_DARKLY_KEY.
  • Delete rorapi/common/features.py and rorapi/v2/tests.py.
  • Remove the duplicate CountryBucket class from rorapi/v2/models.py.
  • Trim the README GRID section and the legacy notes in ARCHITECTURE.md.

Important Technical Details

  • retrieve_organization no longer checks GRID_REMOVED_IDS. That list was empty, so the check never changed a response.
  • CountryBucket remains in rorapi/common/models.py. The copy on rorapi/v2/models.py was unused.
  • rorapi/middleware/ and the single CommonMiddleware entry are unchanged.
  • The merge with dev conflicted only on legacyindexgrid.py, which dev had patched to log str(e). The file stays deleted.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Reviewer, please remember our guidelines:

  • Be humble in the language and feedback you give, ask don't tell.
  • Consider using positive language as opposed to neutral when offering feedback. This is to avoid the negative bias that can occur with neutral language appearing negative.
  • Offer suggestions on how to improve code e.g. simplification or expanding clarity.
  • Ensure you give reasons for the changes you are proposing.
Open in Web Open in Cursor 

Drop the unused GRID/LEGACY_ROR_DUMP settings, LaunchDarkly features
module, empty GRID_REMOVED_IDS lookup, duplicate CountryBucket, unused
v2 Client tests, and the legacy GRID management commands.

Co-authored-by: Joseph Rhoads <jrhoads@users.noreply.github.com>
@jrhoads jrhoads added the quickwin:wave-2 label Sep 25, 2026 — with Cursor
Keep the legacy GRID command deleted. The only overlap with dev was a logging fix in that removed file. Update ARCHITECTURE.md so it no longer documents LaunchDarkly or the deleted commands.

Co-authored-by: Joseph Rhoads <jrhoads@users.noreply.github.com>
@jrhoads
jrhoads requested a lite review from Copilot September 28, 2026 20:01
@jrhoads
jrhoads merged commit 8a9b843 into dev Sep 28, 2026
3 of 4 checks passed
@jrhoads
jrhoads removed the request for review from Copilot September 28, 2026 21:16
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.

2 participants