fix(auth): support running migrations for the first time during auth installation

[MohammadAlhallaq]

fixes #1684

[innocenzi]

Hey, thanks! I don't think hardcoding the migration here is the right approach. I'd probably look into fixing the migration command inside the auth installer instead.

[MohammadAlhallaq]

Hey, thanks! I don't think hardcoding the migration here is the right approach. I'd probably look into fixing the migration command inside the auth installer instead.

yup, I agree, will be done 👍🏻

[MohammadAlhallaq]

Hey, thanks! I don't think hardcoding the migration here is the right approach. I'd probably look into fixing the migration command inside the auth installer instead.

@innocenzi Hey, maybe a combination of checking in the manager and taking action in the installer, keeping the manager as our source of truth for whether the migrations table exists

[MohammadAlhallaq]

@brendt @innocenzi what do you guys think about this one ?

[brendt]

Makes me wonder: why don't we just run migrate:up from within that installer? That way we don't have to check for anything at all. The migration file should have been published in the step before, right?

[MohammadAlhallaq]

Makes me wonder: why don't we just run migrate:up from within that installer? That way we don't have to check for anything at all. The migration file should have been published in the step before, right?

just updated it, maybe something like this ?

[brendt]

No I don't think this is the right approach because migrate:up will run all migrations, and you shouldn't have to manually run another after that. I'll take a look at this myself