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

[js] Added 'ConfigOptionProvider' to provide filenames ending with '.js' in Profile configuration #9646

Merged

Conversation

cweitkamp
Copy link
Contributor

  • Added 'ConfigOptionProvider' to provide filenames ending with '.js' in Profile configuration

Signed-off-by: Christoph Weitkamp [email protected]

@cweitkamp cweitkamp added the enhancement An enhancement or new feature for an existing add-on label Jan 2, 2021
@cweitkamp cweitkamp requested a review from a team as a code owner January 2, 2021 13:48
@cweitkamp cweitkamp changed the title [scale] Added 'ConfigOptionProvider' to provide filenames ending with '.js' in Profile configuration [js] Added 'ConfigOptionProvider' to provide filenames ending with '.js' in Profile configuration Jan 2, 2021
Copy link
Member

@J-N-K J-N-K left a comment

Choose a reason for hiding this comment

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

otherwise looks good

@@ -52,7 +51,7 @@
@NonNullByDefault({})
Copy link
Member

Choose a reason for hiding this comment

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

This annotation is wrong. l. 75 sets it to null. (also above)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Absolutely. I changed it.

@J-N-K
Copy link
Member

J-N-K commented Jan 3, 2021

Can you resolve conflicts?

… Profile configuration

Signed-off-by: Christoph Weitkamp <[email protected]>
Signed-off-by: Christoph Weitkamp <[email protected]>
@cweitkamp cweitkamp force-pushed the feature-js-profile-config-options-provider branch from 839f3a9 to 8b028d3 Compare January 3, 2021 16:46
@cweitkamp
Copy link
Contributor Author

Rebase done.

@J-N-K J-N-K merged commit 25947bf into openhab:main Jan 4, 2021
@J-N-K J-N-K added this to the 3.1 milestone Jan 4, 2021
@cweitkamp cweitkamp deleted the feature-js-profile-config-options-provider branch January 4, 2021 17:47
lucacalcaterra pushed a commit to lucacalcaterra/openhab-addons that referenced this pull request Feb 2, 2021
…js' in Profile configuration (openhab#9646)

* Added ConfigOptionProvider to provide file names ending with '.js' in Profile configuration

Signed-off-by: Christoph Weitkamp <[email protected]>
Signed-off-by: Luca Calcaterra <[email protected]>
thinkingstone pushed a commit to thinkingstone/openhab-addons that referenced this pull request Nov 7, 2021
…js' in Profile configuration (openhab#9646)

* Added ConfigOptionProvider to provide file names ending with '.js' in Profile configuration

Signed-off-by: Christoph Weitkamp <[email protected]>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
…js' in Profile configuration (openhab#9646)

* Added ConfigOptionProvider to provide file names ending with '.js' in Profile configuration

Signed-off-by: Christoph Weitkamp <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants