With the upgrade to PowerFx.Interpreter from 0.2.3-preview to 0.2.6-preview, some implementation details have changed -
DateValue.Value is now deprecated in favor of DateValue.GetConvertedValue(timeZone), and if timeZone is null, it will use the local timezone. This change has been made to factor this in without change in functionality.
The locale parameter has been temporarily removed from PowerFxConfig constructor to unblock us from fixing other compile time errors, and work on this fix and test it in isolation.
The implementation of IUntypedObject interface has been updated per the updated interface.
Most of the checklist items are not applicable as these are in-progress changes being done on an intermediate branch to facilitate parallel efforts on the many compile time errors that this is addressing.
NOTE: This change is being merged into the intermediate feature branch currently. Once all the compile time breaking changes have been addressed and tests are passing we will merge this feature branch into main, for full review of all the compile time fixes at once.
Checklist
[ ] The code change is covered by unit tests. I have added tests that prove my fix is effective or that my feature works
[ ] I have performed end-to-end test locally.
[ ] New and existing unit tests pass locally with my changes
[x] I have commented my code, particularly in hard-to-understand areas
[N/A] I have made corresponding changes to the documentation
Description
With the upgrade to PowerFx.Interpreter from 0.2.3-preview to 0.2.6-preview, some implementation details have changed -
DateValue.Value
is now deprecated in favor ofDateValue.GetConvertedValue(timeZone)
, and iftimeZone
isnull
, it will use the local timezone. This change has been made to factor this in without change in functionality.locale
parameter has been temporarily removed fromPowerFxConfig
constructor to unblock us from fixing other compile time errors, and work on this fix and test it in isolation.IUntypedObject
interface has been updated per the updated interface.Most of the checklist items are not applicable as these are in-progress changes being done on an intermediate branch to facilitate parallel efforts on the many compile time errors that this is addressing.
NOTE: This change is being merged into the intermediate feature branch currently. Once all the compile time breaking changes have been addressed and tests are passing we will merge this feature branch into main, for full review of all the compile time fixes at once.
Checklist