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

Fixed nested arrays #3 #1511

Merged
merged 4 commits into from
Jul 20, 2023
Merged

Conversation

Slonick
Copy link

@Slonick Slonick commented Jul 20, 2023

♻️ Current situation

Nested arrays display only one element.

💡 Proposed solution

To solve the problem, the generated layout and data are matched.

⚙️ Release Notes

The fix is applied only if "fixArrays": true is added to config.schema.json.
For example:
{ "pluginAlias": "examplePluginAlias", "pluginType": "platform", "singular": true, "fixArrays": true, "schema": {...} }

@coveralls
Copy link

Pull Request Test Coverage Report for Build 5608077797

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 43.4%

Totals Coverage Status
Change from base Build 5604417244: 0.0%
Covered Lines: 2113
Relevant Lines: 4507

💛 - Coveralls

@donavanbecker
Copy link
Contributor

@Slonick Is there a reason why we need the config?

@donavanbecker donavanbecker merged commit 7994a83 into homebridge:beta-4.50.5 Jul 20, 2023
20 of 26 checks passed
@Slonick
Copy link
Author

Slonick commented Jul 20, 2023

@Slonick Is there a reason why we need the config?

I still need feedback to fix all the array problems, and then it will be possible to remove and make beta without this flag

@donavanbecker
Copy link
Contributor

@Slonick thanks

@donavanbecker donavanbecker mentioned this pull request Aug 19, 2023
donavanbecker added a commit that referenced this pull request Aug 19, 2023
* Add singular plug-in label (#1458)

* Document the default port/address that you can access the UI in your web browser (#1252)

* Improve translations (#1459)

* Fix capitalization (#1496)

* Update zh-CN.json (#1503)

* Fixed nested arrays settings (#1504)

* Fixed nested arrays settings #2 (#1510)

* Fixed nested arrays;
Fixed lint;

* Removed console.log()

* Fixed nested arrays #3 (#1511)

* Wait for custom UI (#1522)

* Wait for custom UI (#1523)

---------

Co-authored-by: Dave Nicolson <david.nicolson@gmail.com>
Co-authored-by: Adam Coulombe <adamcoulombe187@hotmail.com>
Co-authored-by: Aiden <9078877+aiden0w0@users.noreply.github.com>
Co-authored-by: Artem Kononenko <slonick2012@gmail.com>
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.

None yet

3 participants