Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[releases/24.2] fix bug related to daily no. series lines #1245

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

grobyns
Copy link
Contributor

@grobyns grobyns commented Jun 3, 2024

This pull request backports #1243 to releases/24.2

Fixes AB#537308

<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary <!-- Provide a general summary of your changes -->
The No. Series module tries to use the No Series Line given before
looking for a new line to use. This has the effect that as long as the
usagedate is newer the given line would be used even though a closer
match exists.
This fix makes it so the given line is only used if the starting dates
from the given line and the new line match. If the dates don't match,
the new line is used.

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#537258](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/537258)
@grobyns grobyns requested a review from a team as a code owner June 3, 2024 14:45
@github-actions github-actions bot added this to the Version 24.2 milestone Jun 3, 2024
@grobyns grobyns enabled auto-merge (squash) June 3, 2024 14:50
@grobyns grobyns merged commit d578447 into releases/24.2 Jun 5, 2024
21 of 22 checks passed
@grobyns grobyns deleted the backport/releases/24.2/1243/20240603164459 branch June 5, 2024 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants