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

[blockly] Add new list, item, persistence, text blocks #1549

Merged
merged 9 commits into from
Dec 10, 2022

Conversation

stefan-hoehn
Copy link
Contributor

@stefan-hoehn stefan-hoehn commented Nov 21, 2022

Signed-off-by: Stefan Höhn stefan@andreaundstefanhoehn.de

oh_text_replace: a new text replace block that is missing in the standard text block section
image

oh_taggeditems: retrieve a list of items identified by one or multiple tags
image

oh_list_concatenate: allows to concatenate two lists into a new lists
image

oh_get_persistvalue: add new previousState (new mutating block)

image

fixed type checking of oh_get_persistvalue: requires item name, not item

oh_zdt_fromText: Add time support
image

oh_transformation: fix help url

see some more info and example here: https://community.openhab.org/t/extending-blockly-with-new-openhab-commands/127169/394

Signed-off-by: Stefan Höhn <stefan@andreaundstefanhoehn.de>
@stefan-hoehn stefan-hoehn requested a review from a team as a code owner November 21, 2022 22:06
…nting

Signed-off-by: Stefan Höhn <stefan@andreaundstefanhoehn.de>
@relativeci
Copy link

relativeci bot commented Nov 21, 2022

Job #628: Bundle Size — 15.96MiB (+0.11%).

304816f(current) vs 23c4cd8 main#627(baseline)

Metrics (6 changes)
                 Current
Job #628
     Baseline
Job #627
Initial JS 1.72MiB(-0.32%) 1.72MiB
Initial CSS 608.33KiB 608.33KiB
Cache Invalidation 83.68% 83.7%
Chunks 218 218
Assets 688(+0.15%) 687
Modules 2008(+0.1%) 2006
Duplicate Modules 110(+1.85%) 108
Duplicate Code 1.82%(+0.55%) 1.81%
Packages 133 133
Duplicate Packages 15 15
Total size by type (2 changes)
                 Current
Job #628
     Baseline
Job #627
CSS 856.37KiB 856.37KiB
Fonts 1.08MiB 1.08MiB
HTML 1.21KiB 1.21KiB
IMG 140.74KiB 140.74KiB
JS 9.03MiB (+0.14%) 9.01MiB
Media 295.6KiB 295.6KiB
Other 4.59MiB (+0.12%) 4.58MiB

View job #628 reportView main branch activity

Signed-off-by: Stefan Höhn <stefan@andreaundstefanhoehn.de>
@stefan-hoehn
Copy link
Contributor Author

@ghys Do you mind having a quick look at it, so we can deliver it with the current milestone

@stefan-hoehn
Copy link
Contributor Author

@lolodomo would you be able to review it, so there is a chance that it still can make it into main before the feature freeze?

@ghys
Copy link
Member

ghys commented Dec 2, 2022

I'm still around... I'll review it and other pending PRs this week-end.

@stefan-hoehn
Copy link
Contributor Author

I'm still around... I'll review it and other pending PRs this week-end.

Yannick, I know, I appreciate it. please take care of yourself.! 🤗

Signed-off-by: Stefan Höhn <stefan@andreaundstefanhoehn.de>
Signed-off-by: Stefan Höhn <stefan@andreaundstefanhoehn.de>
Signed-off-by: Stefan Höhn <stefan@andreaundstefanhoehn.de>
@stefan-hoehn
Copy link
Contributor Author

FYI I took the chance and added a few changes lately and a fix all of which are documented above. The reason is that by using the blocks myself I discovered some short comings, so I improved them.

The PR is now stable again and ready to be reviewed.

Signed-off-by: Stefan Höhn <stefan@andreaundstefanhoehn.de>
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.

You're getting good at this, I have just this to moan about but otherwise it's all good.

Signed-off-by: Stefan Höhn <stefan@andreaundstefanhoehn.de>
@stefan-hoehn
Copy link
Contributor Author

@ghys Your change has been applied.

Signed-off-by: Yannick Schaus <github@schaus.net>
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.

There was still missing spaces that I didn't fix in the earlier suggestion, I did it myself.

LGTM thanks.

@ghys ghys merged commit 8d60efb into openhab:main Dec 10, 2022
@ghys ghys added enhancement New feature or request main ui Main UI labels Dec 10, 2022
@ghys ghys added this to the 3.4 milestone Dec 10, 2022
@kaikreuzer kaikreuzer changed the title [blockly] add new list, item, persistence, text blocks [blockly] Add new list, item, persistence, text blocks Dec 11, 2022
@stefan-hoehn stefan-hoehn deleted the blockly_new_blocks2 branch December 12, 2022 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request main ui Main UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants