Skip to content

refactor(field-studio): remove organization-specific migration logic - #6

Merged
vitormattos merged 4 commits into
mainfrom
fix/remove-legacy-production-migration-marker-20261105
Nov 12, 2026
Merged

vitormattos merged 4 commits into
mainfrom
fix/remove-legacy-production-migration-marker-20261105

Conversation

@vitormattos

@vitormattos vitormattos commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

Purpose

Keep Field Studio a generic Akaunting module for defining, validating, storing and auditing custom fields, with no organization-specific configuration or historical migration logic bundled in its code.

Changes

  • Remove the production-specific historical ETL script scripts/production-metadata.php.
  • Remove the legacy free-text parser Domain/ProductionMetadataText.php and its dedicated tests.
  • Remove the Produção Cooperativista migration procedures from docs/operations.md.
  • Clarify the generic module boundary in README.md, docs/development.md, and AGENTS.md.

Company-specific field definitions, policies, discounts, normalization procedures and business rules are not shipped or versioned by Field Studio. Organizations configure their fields at runtime through its UI/API.

Data safety

This is source-code/documentation removal only. No database schema changes, no changes to custom field CRUD, no migration execution, and no deletion or update of stored definitions, values, audit rows, invoices or payments. The Field Studio's own generic database migrations remain intact. Existing installed runtime data is preserved. No production deployment is included.

Validation performed

  • All remaining PHP files: php -l (PHP 8.4 container) — passed.
  • php tests/i18n-check.php — passed (176 catalog strings; 136 references).
  • node --check for all remaining JS tests — passed.
  • module.json and composer.json parse — passed.
  • git diff --check origin/main...HEAD — passed.
  • Repository search for removed migration classes, marker, script, and cooperative-specific field keys — no occurrences.
  • Signed commits with verified DCO.

End-to-end API/browser tests against an isolated Akaunting installation were not executed in this change. CI status must be evaluated at the final HEAD.

Scope exclusions

No alteration of the existing company-specific definitions and values in any installed instance; no changes to the separate #7 select-persistence fix; no migration or data cleanup on Conceição.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos vitormattos changed the title fix(migration): retire transient production marker and clarify field semantics refactor(field-studio): remove organization-specific migration logic Nov 12, 2026
@vitormattos
vitormattos merged commit 3204622 into main Nov 12, 2026
9 checks passed
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.

1 participant