Skip to content

Remove app_execution app#37822

Merged
snopoke merged 2 commits into
masterfrom
sk/remove-app-testing
Jun 12, 2026
Merged

Remove app_execution app#37822
snopoke merged 2 commits into
masterfrom
sk/remove-app-testing

Conversation

@snopoke

@snopoke snopoke commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Product Description

Removes the "Application Test Flows" feature (app testing workflows) and its UI under the Applications tab. This is a completely unused feature.

Technical Summary

Deletes the corehq.apps.app_execution app entirely and drops its two tables (AppWorkflowConfig, AppExecutionLog) via a cleanup migration, following the pattern of the ewsghana removal.

Feature Flag

N/A

Safety Assurance

Safety story

No code outside the app referenced its models or views. Table drops use DROP TABLE IF EXISTS ... CASCADE; the only FK into these tables is between the two dropped tables themselves.

There are no feature flags related to this feature and no data on 'prod' or 'india'.

Automated test coverage

Model registry tests (test_dump_models, test_deletion_models) and corehq/tabs tests pass.

QA Plan

Migrations

  • The migrations in this code can be safely applied first independently of the code. Pay particular attention to backward incompatible operations like RemoveField, RenameField, RemoveConstraint, and others described here that can cause errors when migrations are applied to a live database.

Note: the migration drops the app's tables, so it is not safe to apply while old code is still serving the removed views however since this is an unused feature this is not expected to be an issue.

Rollback instructions

  • This PR can be reverted after deploy with no further considerations

Reverting restores the code but not the dropped tables or their data (of which there isn't any).

Labels & Review

  • Risk label is set correctly
  • The set of people pinged as reviewers is appropriate for the level of risk of the change

🤖 Generated with Claude Code

snopoke and others added 2 commits June 11, 2026 12:35
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@snopoke snopoke added the DON'T REVIEW YET This PR is not ready for review. Commits may be rebased or force-pushed. Don't waste your time. label Jun 11, 2026
@dimagimon dimagimon added the reindex/migration Reindex or migration will be required during or before deploy label Jun 11, 2026
@snopoke snopoke requested review from gherceg and millerdev June 11, 2026 10:45
@snopoke snopoke added product/invisible Change has no end-user visible impact Risk: Medium Change affects files that have been flagged as medium risk. and removed DON'T REVIEW YET This PR is not ready for review. Commits may be rebased or force-pushed. Don't waste your time. labels Jun 11, 2026
@snopoke snopoke marked this pull request as ready for review June 11, 2026 10:46
@gherceg

gherceg commented Jun 11, 2026

Copy link
Copy Markdown
Member

@snopoke could you elaborate on how you know this is an unused feature? I'm not too familiar with it and just want to make sure the motivation and reasoning is documented in the PR.

@snopoke

snopoke commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

@snopoke could you elaborate on how you know this is an unused feature? I'm not too familiar with it and just want to make sure the motivation and reasoning is documented in the PR.

I built the feature in May 2024 (#34487). It is only accessible via the URL (no nav links) and I checked on 'prod' and 'india' for configurations and there are none.

@gherceg gherceg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks. Confirming I also checked 'EU' for any AppWorkflowConfig just to be safe, and there were also none.

@snopoke snopoke merged commit 6c504f0 into master Jun 12, 2026
28 of 29 checks passed
@snopoke snopoke deleted the sk/remove-app-testing branch June 12, 2026 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product/invisible Change has no end-user visible impact reindex/migration Reindex or migration will be required during or before deploy Risk: Medium Change affects files that have been flagged as medium risk.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants