Unused model fields

[jpmckinney]

Are these planned for future use?

• Lender
  • status
  • deleted_at
• User
  • language

I'm also not really sure what Lender.type affects on the frontend.

[yolile]

Lender status, deleted_at

I guess these fields would be useful if we ever need to offboard a Lender

User language

We are not using this one at all, as we select the language depending on the front-end request

I'm also not really sure what Lender.type affects on the frontend.

It is part of the lender on boarding form, but we are not using it for something in particular at the moment

[jpmckinney]

Ok, I'll just document them as unused. In future, we might just drop them in a migration.

I also notice that it's a bit weird that Lender.logo_filename refers to a file that is stored in git in credere-frontend... It would make more sense to store the logo as bytes in the DB.

[yolile]

I also notice that it's a bit weird that Lender.logo_filename refers to a file that is stored in git in credere-frontend... It would make more sense to store the logo as bytes in the DB.

Of course, see #175