Feat/remove-disabled-columns - Githubissues

jdenoc / money-tracker

income/expense tracker with receipt retention

MIT License

5 stars 0 forks source link

Feat/remove-disabled-columns #530

Closed jdenoc closed 1 year ago

jdenoc commented 1 year ago

Modified the Account model

Removed the accounts.disable column and are relying entirely on the accounts.disabled_stamp to indicate if an account is disabled using the built-in Laravel soft-delete functionality.
Changed the Account->disabled attribute to Account->active.
Removed Account::find_account_with_types()

Modified the AccountType model

Removed the account_types.disabled column and are relying entirely on the account_types.disabled_stamp to indicate if an account is disabled using the built-in Laravel soft-delete functionality.
Refactored the routes for /api/account* endpoints.

Modified the Institution model

Modified the institutions table and object, to handle disabling/enabling an institution.
Removed Institution::find_institution_with_accounts.

Modified the Entry model

Removed Entry::get_entry_with_tags_and_attachments().

Tests

Added functionality to allow the tests to import the latest database snapshot prior to running a test suite.
Converted Tests\Browser\SettingsBase to be an abstract class.
Renamed methods and variables in the Settings*Test classes.
Moved SettingsAccountsTest::testDisablingOrRestoringAccount to SettingsBase.

Misc.

Added constants for cache keys.
Added OptimizedAppCheck health check.
Fixed a bug where renaming database table columns in migrations cause the column type to change from timestamp to datetime.

jdenoc commented 1 year ago

Replacing in favour of separate PRs.