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

[PORT] [Expression] Add more timex builtin functions #2187

Merged
merged 3 commits into from
May 6, 2020
Merged

Conversation

Danieladu
Copy link
Contributor

@Danieladu Danieladu commented May 6, 2020

Fixes #2165
Fixes #2169

Description

Add 7 timex builtin functions:

  • isDefinite
  • isTime
  • isDuration
  • isDate
  • isTimeRange
  • isDateRange
  • isPresent

@Danieladu Danieladu requested a review from cosmicshuai May 6, 2020 08:49
@Danieladu Danieladu changed the title [PORT] [Expression] Add more timex builtin function [PORT] [Expression] Add more timex builtin functions May 6, 2020
@coveralls
Copy link

coveralls commented May 6, 2020

Pull Request Test Coverage Report for Build 128694

  • 64 of 64 (100.0%) changed or added relevant lines in 2 files are covered.
  • 9 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.0008%) to 79.732%

Files with Coverage Reduction New Missed Lines %
libraries/botframework-streaming/src/webSocket/webSocketTransport.ts 2 80.65%
libraries/botframework-streaming/src/webSocket/webSocketServer.ts 7 66.67%
Totals Coverage Status
Change from base Build 128481: 0.0008%
Covered Lines: 12533
Relevant Lines: 14992

💛 - Coveralls

@cleemullins cleemullins merged commit c3f7e33 into master May 6, 2020
@cleemullins cleemullins deleted the hond/timex branch May 6, 2020 16:37
boydc2014 pushed a commit that referenced this pull request May 7, 2020
* add more timex function

* fix typo

* fix typo
boydc2014 added a commit that referenced this pull request May 7, 2020
* [PORT] [Expression] Add more timex builtin functions (#2187)

* add more timex function

* fix typo

* fix typo

* Add formatNumber, parseEpoch and parseTicks. (#2198)

* Add formatNumber, parseEpoch and parseTicks.

* Switch to ESNext like other packages.
Switch ticks to bigInt.
Add tests.

Co-authored-by: Shuai Wang <[email protected]>

Co-authored-by: Hongyang Du (hond) <[email protected]>
Co-authored-by: Chris McConnell <[email protected]>
Co-authored-by: Shuai Wang <[email protected]>
Stevenic added a commit that referenced this pull request May 8, 2020
* Add Dialog / PageView telemetry (#2145)

* Update app insights package version

* Add PageView logging

* move TelemetryView helper to botTelemetryClient.ts

* remove telemetry Extensions class, harden helper method, add tests

Co-authored-by: Steven Gum <[email protected]>

* move botbuilder-lg and adaptive-expressions out of preview (#2190)

* move botbuilder-lg and adaptive-expressions out of preview

* correct set-version script in preview packages

* restore preview packages to using Version instead of PreviewPackageVersion

* Fixed issues with AdaptiveSkillDialog (#2196)

and added unit tests

* [Cherry-pick] timex and formatX functions (#2200)

* [PORT] [Expression] Add more timex builtin functions (#2187)

* add more timex function

* fix typo

* fix typo

* Add formatNumber, parseEpoch and parseTicks. (#2198)

* Add formatNumber, parseEpoch and parseTicks.

* Switch to ESNext like other packages.
Switch ticks to bigInt.
Add tests.

Co-authored-by: Shuai Wang <[email protected]>

Co-authored-by: Hongyang Du (hond) <[email protected]>
Co-authored-by: Chris McConnell <[email protected]>
Co-authored-by: Shuai Wang <[email protected]>

* Updated packages to target ES6 (#2204)

* Switch adaptive-expressions to es6 (#2205)

* Drop back to es5 and add bigint support.

* Fix test script.

* Fix test script.

* Remove adaptive.

* Remove space.

* Remove changes to package.json.

* update adaptive-expressions target to es5 for initial testing

* Push es5.

Co-authored-by: stevengum <[email protected]>

* Renamed AdaptiveSkillDialog to BeginSkillDialog (#2212)

* Renamed BeginSkill action again (#2215)

* Renamed BeginSkill action again

* update component registration

Co-authored-by: Gary Pretty <[email protected]>
Co-authored-by: Steven Gum <[email protected]>
Co-authored-by: Dong Lei <[email protected]>
Co-authored-by: Hongyang Du (hond) <[email protected]>
Co-authored-by: Chris McConnell <[email protected]>
Co-authored-by: Shuai Wang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants