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

No. Series: Ability to extend filters on finding No. Series Lines when getting new numbers #2361

Merged
merged 12 commits into from
Nov 26, 2024

Conversation

PeterDurrer
Copy link
Contributor

@PeterDurrer PeterDurrer commented Nov 14, 2024

Summary

Add Event for No. Series

Work Item(s)

Fixes #1362

Fixes AB#558102

@PeterDurrer PeterDurrer requested a review from a team as a code owner November 14, 2024 08:07
@github-actions github-actions bot added AL: Business Foundation From Fork Pull request is coming from a fork labels Nov 14, 2024
@PeterDurrer PeterDurrer changed the title https://github.com/microsoft/BCApps/issues/1362#issue-2360774022 No. Series: Ability to extend filters on finding No. Series Lines when getting new numbers Nov 14, 2024
@PeterDurrer PeterDurrer reopened this Nov 14, 2024
@PeterDurrer
Copy link
Contributor Author

PeterDurrer commented Nov 14, 2024

@PeterDurrer please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree company="Logico Solutions AG"

AndreasMoth
AndreasMoth previously approved these changes Nov 14, 2024
Copy link
Contributor

@AndreasMoth AndreasMoth left a 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.

@JesperSchulz JesperSchulz added the Integration GitHub request for Integration area label Nov 14, 2024
@PeterDurrer
Copy link
Contributor Author

@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.
@dominicstarkl can you check whether these adjustments provide you with the required code sections that you wanted in issue #1362

@PeterDurrer PeterDurrer requested a review from NKarolak November 20, 2024 10:01
AndreasMoth
AndreasMoth previously approved these changes Nov 21, 2024
grobyns
grobyns previously approved these changes Nov 21, 2024
@PeterDurrer
Copy link
Contributor Author

@PeterDurrer please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree company="Logico Solutions AG"

@JesperSchulz JesperSchulz added the Linked Issue is linked to a Azure Boards work item label Nov 21, 2024
@github-actions github-actions bot added this to the Version 26.0 milestone Nov 21, 2024
@JesperSchulz JesperSchulz enabled auto-merge (squash) November 21, 2024 10:10
auto-merge was automatically disabled November 21, 2024 14:02

Head branch was pushed to by a user without write access

@PeterDurrer PeterDurrer dismissed stale reviews from grobyns and AndreasMoth via 4f1174f November 21, 2024 14:02
@JesperSchulz JesperSchulz merged commit bc36899 into microsoft:main Nov 26, 2024
20 of 21 checks passed
@PeterDurrer
Copy link
Contributor Author

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?

@JesperSchulz
Copy link
Contributor

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.

JesperSchulz pushed a commit that referenced this pull request Nov 28, 2024
JesperSchulz added a commit that referenced this pull request Nov 28, 2024
… 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]>
@dominicstarkl
Copy link

@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.
We want to let the user choose from a no. series entry (LookupMode).
If this procedure would be public we could avoid code copying.

Thanks in advance for your appreciated reply!

Regards
Dominic

@grobyns
Copy link
Contributor

grobyns commented Dec 2, 2024

@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.

@dominicstarkl
Copy link

@grobyns
Hi Gert

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..
In DrillDown procedure we cannot specify the page behavior. We need to set LookUp mode to a page variable and run the page in our own.

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.
As I can see this could be done without any interferences.

What do you think about it?

Regards
Dominic

@dominicstarkl
Copy link

@grobyns Now I have just seen that this whole Codeunit is internal. So we should place the public access somewhere else.

@grobyns
Copy link
Contributor

grobyns commented Dec 3, 2024

@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.

@dominicstarkl
Copy link

@grobyns Thanks Gert, I'll have a look at it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AL: Business Foundation From Fork Pull request is coming from a fork Integration GitHub request for Integration area Linked Issue is linked to a Azure Boards work item
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No. Series: Ability to extend filters on finding No. Series Lines when getting new numbers
6 participants