-
Notifications
You must be signed in to change notification settings - Fork 175
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
No. Series: Ability to extend filters on finding No. Series Lines when getting new numbers #2361
Conversation
…eplaces the backwards compatible Event
…eplaces the backwards compatible Event
@microsoft-github-policy-service agree company="Logico Solutions AG" |
src/Business Foundation/App/NoSeries/src/Single/NoSeriesImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Single/NoSeriesImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
…eplaces the backwards compatible Event
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Single/NoSeriesImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just need the other event to call the new GetNoSeriesLineFilters procedure as well.
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Single/NoSeries.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Single/NoSeries.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Single/NoSeriesImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
…he backwards compatible Event
@grobyns and @AndreasMoth There was some feedback and I have now tried to implement it accordingly. Please check the code again, thank you very much for this intensive review. |
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
|
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
src/Business Foundation/App/NoSeries/src/Setup/NoSeriesSetupImpl.Codeunit.al
Show resolved
Hide resolved
…he backwards compatible Event
Head branch was pushed to by a user without write access
4f1174f
Dear @JesperSchulz , where can I find out from which BC version this will be available? I saw that 26.0 was set under milestones, but that will still be quite a while away. How do I find out whether this will be available in the next CU? |
By default, the next release "vehicle" is the upcoming major - which in this case is 26.0. Upon request we happily backport for release with the upcoming minor though! I will set things in motion right away 😊 Expected release will then be 25.2. |
…n getting new numbers (#2361) #### Summary Add Event for No. Series #### Work Item(s) Fixes #1362 Fixes [AB#558102](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/558102)
… on finding No. Series Lines when getting new numbers (#2416) <!-- 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 --> Backport of #2361 #### 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#558934](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/558934) Co-authored-by: Peter Durrer <[email protected]>
@PeterDurrer @grobyns @AndreasMoth Thanks for your great implementation of this extensibility request - it fully meets our customers requirements 👍 Would it be possible to change this procedure from local to public?
It would help us to invoke the already public procedure DrillDown() without automatically running the no. series page. Thanks in advance for your appreciated reply! Regards |
@dominicstarkl I don't quite understand what you're trying to do. Also, the drilldown method you point to is not public, it's internal. |
@grobyns As far as I can see there is no access modifier in procedure DrillDown. Could you show me your source code? We want to select a No. Series Line from the possible No. Series Lines that show when you drill down from setup.. Of course we could copy the code but it would be more future proof when this procedure was public and not local as it is right now. What do you think about it? Regards |
@grobyns Now I have just seen that this whole Codeunit is internal. So we should place the public access somewhere else. |
@dominicstarkl I will take a look at the ask. I'm not convinced it will work as you expect. Regardless, you should open a new issue for this. This pull request is closed and we'll need a new issue to handle this follow up case. |
@grobyns Thanks Gert, I'll have a look at it |
Summary
Add Event for No. Series
Work Item(s)
Fixes #1362
Fixes AB#558102