Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge cards and accounts table into wallets #41

Open
2 of 7 tasks
dmitryrck opened this issue Jun 15, 2021 · 0 comments
Open
2 of 7 tasks

Merge cards and accounts table into wallets #41

dmitryrck opened this issue Jun 15, 2021 · 0 comments

Comments

@dmitryrck
Copy link
Owner

dmitryrck commented Jun 15, 2021

  • Rename accounts to wallets
  • Migrate all cards to wallets (Using SQL and Rails migration) Remove the card feature #70
  • Update movements table to use only wallets (chargeablewallet)
  • Drop cards table

Notes

  • There is a custom error message introduced by 7c512a0 , it is worth removing it and leaving only the ActiveAdmin way.
  • Be careful when updating movements table, remember that the id on accounts and cards both start with 1, it might be worth adding two temporary columns to help migrate on wallets. For example: card_id (can be null) and is_card (default false, when migrating cards to wallets set as true)
  • There is one feature for paying card debts in the app, make sure it is working after this migration
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

No branches or pull requests

1 participant