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

[MainUI] Update JS Scripting tern definitions #1552

Merged
merged 7 commits into from
Dec 4, 2022

Conversation

florian-h05
Copy link
Contributor

@florian-h05 florian-h05 commented Nov 24, 2022

The tern definitions are now up-to-date for version 3.0.0 of openhab-js.

Fixes #1550.

Description

The tern definitions for JS Scripting were updated in #1421 to address the openhab-js release 2.0.0.

Since then, we had numerous improvements and changes to the openhab-js library: openhab/openhab-js@v2.0.0...v3.0.0.

This updates the JS Scripting tern definitions:

  • ItemHistory: Add new methods
  • ScriptExecution: Remove createTimerWithArgument & Add Timer
  • actions: Add thingActions
  • Add Item semantics
  • Add Transformation actions

Testing

I have tested it, all changes work fine.

@florian-h05 florian-h05 requested a review from a team as a code owner November 24, 2022 21:59
@relativeci
Copy link

relativeci bot commented Nov 24, 2022

Job #614: Bundle Size — 15.94MiB (~+0.01%).

c03bf41(current) vs 781f4a2 main#613(baseline)

Metrics (no changes)
                 Current
Job #614
     Baseline
Job #613
Initial JS 1.71MiB 1.71MiB
Initial CSS 608.33KiB 608.33KiB
Cache Invalidation 83.69% 83.69%
Chunks 218 218
Assets 687 687
Modules 2008 2008
Duplicate Modules 110 110
Duplicate Code 1.82% 1.82%
Packages 133 133
Duplicate Packages 15 15
Total size by type (2 changes)
                 Current
Job #614
     Baseline
Job #613
CSS 856.37KiB 856.37KiB
Fonts 1.08MiB 1.08MiB
HTML 1.21KiB 1.21KiB
IMG 140.74KiB 140.74KiB
JS 9.02MiB (~+0.01%) 9.02MiB
Media 295.6KiB 295.6KiB
Other 4.58MiB (~+0.01%) 4.58MiB

View job #614 reportView main branch activity

... as they are not used/referenced from the rest of the tern defs and not used in JS Scripting as well.

Signed-off-by: Florian Hotze <[email protected]>
@florian-h05 florian-h05 changed the title [WIP] [MainUI] Update JS Scripting tern definitions [MainUI] Update JS Scripting tern definitions Dec 3, 2022
@florian-h05
Copy link
Contributor Author

@ghys This is ready for review now and should make it into 3.4.0!

Copy link
Member

@ghys ghys left a comment

Choose a reason for hiding this comment

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

Thanks, I suppose this susperses former submissions and apologies for the delay.
Having an autocompletion in the UI that matches what's available in the library is VERY important to me so I'm happy to merge this and let me know if there are last-minute things to improve.

@ghys ghys merged commit 6e3aad1 into openhab:main Dec 4, 2022
@florian-h05 florian-h05 deleted the jsscripting-tern-defs branch December 4, 2022 23:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update JS Scripting tern defs for latest openhab-js release
2 participants