Skip to content

Drop Duplicate Exclusion doctype and remove Party workspace references#25

Merged
Sendipad merged 2 commits intodevelopfrom
codex/analyze-code-for-bugs-and-anti-patterns
Feb 22, 2026
Merged

Drop Duplicate Exclusion doctype and remove Party workspace references#25
Sendipad merged 2 commits intodevelopfrom
codex/analyze-code-for-bugs-and-anti-patterns

Conversation

@Sendipad
Copy link
Copy Markdown
Owner

Motivation

  • The Duplicate Exclusion doctype is deprecated and should be removed from the codebase to avoid exposing a stale UI surface and duplicated functionality.
  • Clean up the Party workspace so users no longer see links/shortcuts to the removed doctype and the workspace JSON remains consistent with repository serialization style.

Description

  • Deleted the doctype implementation files under uph/party/doctype/duplicate_exclusion/ including duplicate_exclusion.json, duplicate_exclusion.py, duplicate_exclusion.js, test_duplicate_exclusion.py, and __init__.py.
  • Removed all Duplicate Exclusion entries from the Party workspace uph/party/workspace/party/party.json shortcuts and links and restored the content serialization format to the repository style.
  • Left historical migration patch references (e.g. uph.patches.migrate_duplicate_exclusion_to_party_issue) and localization strings unchanged to preserve migration history and avoid altering patches that are defensive when the doctype is absent.

Testing

  • Ran python -m py_compile uph/party/page/data_quality_dashboard/data_quality_dashboard.py and it completed successfully.
  • Validated workspace JSON with python -m json.tool uph/party/workspace/party/party.json >/dev/null and this validation succeeded.
  • Captured a browser screenshot of the Party workspace after the change (artifact path shown in commit metadata) as an automated visual check.

Codex Task

@Sendipad Sendipad merged commit 4dd8a39 into develop Feb 22, 2026
6 checks passed
@Sendipad Sendipad deleted the codex/analyze-code-for-bugs-and-anti-patterns branch February 22, 2026 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant