[ENH] Add UseMySql parameter about elsa-core HOT 6 CLOSED

To allow MySQL to use AutoDetect and a hardcoded version via overload, you can refer to the commit mentioned in the question. This commit introduces changes that enable the use of an overload for specifying the MySQL version explicitly or allowing the system to auto-detect it.

The commit comment linked in the question provides a discussion around this feature, where the contributors to the Elsa project discuss the implementation details. You can view the changes made in the commit to understand how to implement this in your own project.

Please note that you should always ensure that you are using compatible versions of the packages and that your project's dependencies are correctly configured to avoid any conflicts or issues.

If you need further clarification or assistance, please provide more context or specific questions regarding the implementation.

This comment was generated by Glime.

from elsa-core.

Hi. What do you think about adding method with ServerVersion parameter and use old method with autodetect?

public static DbContextOptionsBuilder UseElsaMySql(this DbContextOptionsBuilder builder, Assembly migrationsAssembly, string connectionString, ElsaDbContextOptions? options = default, Action<MySqlDbContextOptionsBuilder>? configure = default) =>
        UseElsaMySql(builder, migrationsAssembly, connectionString, ServerVersion.AutoDetect(connectionString), options, configure);

public static DbContextOptionsBuilder UseElsaMySql(this DbContextOptionsBuilder builder, Assembly migrationsAssembly, string connectionString, ServerVersion serverVersion, ElsaDbContextOptions? options = default, Action<MySqlDbContextOptionsBuilder>? configure = default) =>
    builder
            .UseElsaDbContextOptions(options)
            .UseMySql(connectionString, serverVersion, db =>
            {
                db
                    .MigrationsAssembly(options.GetMigrationsAssemblyName(migrationsAssembly))
                    .MigrationsHistoryTable(options.GetMigrationsHistoryTableName(), options.GetSchemaName())
                    .SchemaBehavior(MySqlSchemaBehavior.Ignore);

                configure?.Invoke(db);
            });

If it's appropriate solution I can open PR today.

from elsa-core.

There is already PR to reverse changes about autodetect (#5621). May be author can add overload that I suggested.

from elsa-core.

@4lexKislitsyn Makes sense. @raymonddenhaan do you want to update your PR, or shall we ask @4lexKislitsyn to open a new one?

from elsa-core.

@4lexKislitsyn, @sfmskywalker I Updated the PR with the overload

from elsa-core.

Perfect, thanks @raymonddenhaan & @4lexKislitsyn !

from elsa-core.