feat: Made ViewController to accept queries for external parameters (breaking change)

[mnasyrov]

No description provided.

[coveralls]

Pull Request Test Coverage Report for Build 4065216494

• 12 of 12 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 100.0%

---

Totals
Change from base Build 4037653106:	0.0%
Covered Lines:	363
Relevant Lines:	363

---

💛 - Coveralls

[chartyom]

Hi, Mikhail.
This PR is great!

When I was checking, I found one case allows a lot of executions

 PASS  packages/rx-effects-react/src/mvc.test.tsx (8.931 s)
  Console

    console.log
      object [ 1, 2 ]

      at packages/rx-effects-react/src/mvc.tsx:49:13

    console.log
      object [ 4, 10 ]

      at packages/rx-effects-react/src/mvc.tsx:49:13

    console.log
      object [ 4, 10 ]

      at packages/rx-effects-react/src/mvc.tsx:49:13

You can use the spread to memorize arguments

  useEffect(() => {
    params.forEach((value, index) => stores[index].set(value));
  }, [...params, stores]);

And in this case it will be like this

 PASS  packages/rx-effects-react/src/mvc.test.tsx (8.164 s)
  Console

    console.log
      object [ 1, 2 ]

      at packages/rx-effects-react/src/mvc.tsx:49:13

    console.log
      object [ 4, 10 ]

      at packages/rx-effects-react/src/mvc.tsx:49:13

[mnasyrov]

@chartyom Thank you. Please take a look on the last commit, I used a reference to keep stores out of rendering.

[chartyom]

@mnasyrov perfect!