Enhancement: First field in repeating group as delimiter

[andreydp]

Fixes #675

[chrjohn]

Thanks for the PR! 👍
Could you please also add the configuration to the validation section of https://github.com/quickfix-j/quickfixj/blob/master/quickfixj-core/src/main/doc/usermanual/usage/configuration.html

Thanks in advance

[andreydp]

Thanks, I've added changes to the documentation and also added session parameter.

[chrjohn]

Could you please check my comments. Thank you. :)

[andreydp]

Thanks for the review, pushed changes to the branch.

[noblemaster]

We are having the same problem with repeating group delimiters. It would be nice to have this merged.

[noblemaster]

@andreydp We are having the same problem. Would it be OK if I rework your pull request for the new 3.0.0 release? I'll get started beginning of next week in case I don't hear back.

[andreydp]

Hi @noblemaster. I can adapt my changes for 3.0.0 release in the next few days.

[noblemaster]

@andreydp Thanks! I can assist/review as needed. 👍

[andreydp]

@noblemaster working on the changes, will push soon, sorry for the delay.

[andreydp]

@noblemaster I've merged latest master

[noblemaster]

The change looks good! I have nothing to add. 👍

I would approve it. That's a nice addition. @chrjohn I don't have the powers to approve, so you will have to have a look too. Thanks.

[philipwhiuk]

Will review tomorrow - have similar stuff in #876 which I'm hoping to work on this month.

[chrjohn]

This feature only works when unordered group fields are not validated. Does it make sense to throw an error or generate a warning when this feature is enabled and unordered group fields are validated?
@andreydp @noblemaster @philipwhiuk

[andreydp]

I've added a soft warning in createValidationSettings. Please review/merge.

[noblemaster]

@chrjohn Could you review the changes. Looks generally fine to me.

[chrjohn]

Closing and reopening to trigger github actions which weren't active when this PR was created.
Edit: doesn't seem to work, don't know why Java CI actions are not running...

[chrjohn]

Now it is working, don't know why, though.

[noblemaster]

@andreydp Can you merge your pull request? :)

[chrjohn]

No, he can't. :)
I will do it this week.

[noblemaster]

Great! Is there a planned v3.0.0 release date as well?

[chrjohn]

2025 🚀 😆
I will really try to do this by end of February the latest. But as I said somewhere else it really would help if I knew of users that are testing their apps with the 3.0.0 snapshot. Unit and acceptance tests are one thing but real life tests are another.

[andreydp]

@chrjohn, @noblemaster thank you for review!

[chrjohn]

Again, closing to get all Java CI checks...