Skip to content

Add evaluation company work date option - #11079

Draft
Nikola Kukrika (nikolakukrika) wants to merge 2 commits into
mainfrom
nikolak/use-today-as-workdate
Draft

Add evaluation company work date option#11079
Nikola Kukrika (nikolakukrika) wants to merge 2 commits into
mainfrom
nikolak/use-today-as-workdate

Conversation

@nikolakukrika

@nikolakukrika Nikola Kukrika (nikolakukrika) commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add Use Today as a Workdate to Company Information for evaluation companies
  • use Today instead of the latest G/L entry posting date when enabled
  • preserve existing behavior for evaluation companies when disabled and for regular companies
  • propagate the Company Information field and page control to dependent country layers
  • add behavior and visibility tests

Testing

  • Not run (per request)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: d65116f3-99f0-4e6e-8de8-9cfc17dbc45e
@github-actions github-actions Bot added the Team: Integrations GitHub request for Integrations area label Sep 4, 2026
begin
if CompanyInformationMgt.IsDemoCompany() then begin
CompanyInformation.SetLoadFields("Demo Company", "Use Today as a Workdate");
if CompanyInformation.Get() and CompanyInformation."Demo Company" then begin

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of doing this can we add an overload to CompanyInformationMgt.IsDemoCompany that returns the company information, like CompanyInformationMgt.IsDemoCompany(CompanyInformation)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressed in 58c950a. Added an internal CompanyInformationMgt.IsDemoCompany(var CompanyInformation) overload, kept the existing no-argument API delegating to it, and updated LogInManagement to use the overload with the required load fields.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Could not find a linked work item. Please link one in either of these ways: (1) link an ADO work item using the pattern 'AB#' followed by the work item number - you may use the 'Fixes' keyword to automatically resolve it when the pull request is merged, e.g. 'Fixes AB#1234'; or (2) link a GitHub issue of type 'Task' using the pattern 'Fixes #' followed by the issue number, e.g. 'Fixes #1234'.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: d65116f3-99f0-4e6e-8de8-9cfc17dbc45e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team: Integrations GitHub request for Integrations area

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant