Add default value to row.Getxxx() methods

[PhilippDehner]

I would like to use the extension methods in Reqnroll/Assist/RowExtensionMethods.cs with a default value if the column not exists.

  public static string GetString(this DataTableRow row, string id, string? defaultValue = null)
        {
            return AValueWithThisIdExists(row, id)
                       ? row[id]
                       : defaultValue;
        }

[clrudolphi]

Thank you for the suggestion @PhilippDehner. This would be a good candidate for someone's first contribution. Would you like to submit a PR for this?

[PhilippDehner]

Is there a documentation how to write codes? Or for the first step I only will edit the implementation.

Who creates an issue for that?

[clrudolphi]

Please follow the instructions in CONTRIBUTING.md. You've already begun by raising this Discussion Idea.
When you have code that you'd like to submit, create a pull request.

[304NotModified]

Please also note that adding an optional parameter is a runtime breaking change. So create overloads please :)

[304NotModified]

I think you only need one extra overload when using [NotNullIfNotNull]

But is that's hard with platform support, 2 extra overloads would be fine.

Please don't forget the unit tests for all methods.

[gasparnagy]

Small note: We are in a special period: Currently we are planning Reqnroll v4 so we can have breaking changes as well as long as v4 is not released.

[PhilippDehner]

Small note: We are in a special period: Currently we are planning Reqnroll v4 so we can have breaking changes as well as long as v4 is not released.

It can have breaking changes but should not have or it doesn't matter if there are more breaking changes?

[clrudolphi]

It can have breaking changes (up until the date that v4 is released)

[304NotModified]

I've created this with Claude #1082