FlowFuse / nr-launcher

A Laucher for Node-RED that allows settings to be loaded at startup.
Apache License 2.0
4 stars 6 forks source link

Allow the parent window to configure and manage navigation requests #224

Closed cstns closed 4 months ago

cstns commented 4 months ago

Allow the parent window to configure and manage navigation requests when the NR editor resides in an embedded context/iframe

Description

Related Issue(s)

Checklist

Labels

@closes https://github.com/FlowFuse/nr-launcher/issues/223

cstns commented 4 months ago

Some feedback / input would be appreciated on the interception of the logo click event

hardillb commented 4 months ago

If we are confident that this is it for changes to the theme, then yes, bump the inner package.json.

@sherbastian The top level nr-launcher package.json will get bumped by the release script I mentioned on our call earlier. I will try to remember to point it out in the release huddle on Thursday.

cstns commented 4 months ago

Reverted the v bump, the PR should be good to go. It can be released independently of #3656 without any side effects

cstns commented 4 months ago

@Steve-Mcl, should I do it manually?

Steve-Mcl commented 4 months ago

@Steve-Mcl, should I do it manually?

Yes, just update the value in this PR. The inner package is not published and has no automations.