The table is missing indexes

[ash-jc-allen]

The table is missing indexes

[ash-jc-allen]

Hey @andrey-helldar, thanks for flagging this up. I mainly intended for the package's migrations to only create the structure of the tables so that the dev using the package can have more granular control themselves. Of course, if you wish to add indexes to the tables, you can publish the migrations and add them yourself manually 😄

[andrey-helldar]

It is strange for me to force other developers to make changes to the functionality of the package.

In addition, there are many newbies in our industry who don't even know how indexes work.

That is why I suggested changing the package code.

The indexes I found are critical when searching and must be, but now the responsibility for creating them needs to be shifted to others...

However, good luck!

[ash-jc-allen]

I'll do a bit of reading into this and check across other packages as well to see what the convention is for this type of thing. If it's looking like some of the other mainstream packages also add indexes to their migrations, I think it'd be good to get them added.

If you'd like to contribute, feel free to make a PR 😄

[andrey-helldar]

I trust you 😁