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.x] fixes an issue where a subscriber can no longer change the result from GetNextNo or PeekNextNo. The fix is for obsolete code only. #1460

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

grobyns
Copy link
Contributor

@grobyns grobyns commented Jul 2, 2024

This pull request backports #1446 to releases/24.x

Fixes AB#540017

…m GetNextNo or PeekNextNo. The fix is for obsolete code only. (#1446)

<!-- 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 -->
fixes an issue where a subscriber can no longer change the result from
GetNextNo or PeekNextNo. The fix is for obsolete code only.
#1268
microsoft/ALAppExtensions#26721

#### 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#539238](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/539238)
@grobyns grobyns requested a review from a team as a code owner July 2, 2024 08:25
@grobyns grobyns enabled auto-merge (squash) July 2, 2024 08:29
@github-actions github-actions bot added this to the Version 24.4 milestone Jul 2, 2024
@grobyns grobyns merged commit e7ee0fc into releases/24.x Jul 2, 2024
20 of 21 checks passed
@grobyns grobyns deleted the backport/releases/24.x/1446/20240702102515 branch July 2, 2024 09:05
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