🐛 [Bug] Creating migration for postgresql doesn't work about goravel

[hwbrzzl]

Hi, thanks for your issue. Actually, the database/migrations/20221001000000_create_users_table.up.sql file is only applicable to Mysql. If you want to use Postgresql, you need to change the default connection in the config/database.go file and optimize the *.sql file applicable to Postgresql, such as:

CREATE TABLE users(
  id int PRIMARY KEY NOT NULL,
  name VARCHAR
);

If you have any other questions, please contact me, thanks.

[sujit-baniya]

@hwbrzzl I did change the default connection to postgresql. So once the file is generated, I need to change the content of sql for postgresql?

[hwbrzzl]

Yup, the template that creating by go run . artisan make:migration create_users_table is only applicable to Mysql now. We can generate a Postgresql template based on the default connection in the config/database.go file in the future, but unfortunately, we have to change the .sql file manually now.

If you're interested, I'm very glad that you can submit a PR for that. 👍

[sujit-baniya]

I have done something similar for migration here https://github.com/sujit-baniya/db/blob/master/migrations/command_new.go

[hwbrzzl]

Great, stared. 👍

[sujit-baniya]

@hwbrzzl Do you think we could replace current go-migrate/migrate with above?

[hwbrzzl]

If you can perfect your README, I'm glad to read it first.