749b1db7c7
Bump @testing-library/jest-dom from 5.15.0 to 5.15.1 in /taker-frontend
Bumps [@testing-library/jest-dom ](https://github.com/testing-library/jest-dom ) from 5.15.0 to 5.15.1.
- [Release notes](https://github.com/testing-library/jest-dom/releases )
- [Changelog](https://github.com/testing-library/jest-dom/blob/main/CHANGELOG.md )
- [Commits](https://github.com/testing-library/jest-dom/compare/v5.15.0...v5.15.1 )
---
updated-dependencies:
- dependency-name: "@testing-library/jest-dom"
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
3 years ago
10b0445540
Bump ndarray from 0.15.3 to 0.15.4
Bumps [ndarray](https://github.com/rust-ndarray/ndarray ) from 0.15.3 to 0.15.4.
- [Release notes](https://github.com/rust-ndarray/ndarray/releases )
- [Changelog](https://github.com/rust-ndarray/ndarray/blob/master/RELEASES.md )
- [Commits](https://github.com/rust-ndarray/ndarray/compare/0.15.3...0.15.4 )
---
updated-dependencies:
- dependency-name: ndarray
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
3 years ago
a7917f1e86
Merge #694
694: Clean up the test harness before allowing config tailoring r=klochowicz a=klochowicz
Some of the tests will use the same parameters as production code to reproduce
and fix issues such as heartbeat timeout during contract setup.
Co-authored-by: Mariusz Klochowicz <mariusz@klochowicz.com>
3 years ago
fde200eee7
Merge #695
695: Remove `maker_cfd::CfdAction` enum r=thomaseizinger a=thomaseizinger
Originally we thought we need this enum because we would need to use
`MessageChannel`s from the rocket layer. But we can actually fully
qualify the address without issues.
We also introduce `xtra_productivity` to remove some of the indirection
that is happening in this file.
Co-authored-by: Thomas Eizinger <thomas@eizinger.io>
3 years ago
07c79b8396
Remove `maker_cfd::CfdAction` enum
Originally we thought we need this enum because we would need to use
`MessageChannel`s from the rocket layer. But we can actually fully
qualify the address without issues.
We also introduce `xtra_productivity` to remove some of the indirection
that is happening in this file.
3 years ago
a8d1647d69
Clean up the test harness before allowing config tailoring
Some of the tests will use the same parameters as production code to reproduce
and fix issues such as heartbeat timeout during contract setup.
3 years ago
52640dd19a
Merge #679
679: Refactor `useAsync` usages into `usePostRequest` hook r=thomaseizinger a=thomaseizinger
Co-authored-by: Thomas Eizinger <thomas@eizinger.io>
3 years ago
2c154f5854
Merge #690
690: Move settlement oracle event id into the Dlc r=da-kami a=da-kami
Oracle event-id and Dlc have to exist together, one without the other is incomplete, hence we move the event id inside.
Note that the ID is actually already know to the Dlc as part of the `cet` hashmap, but this was not cleaned up in this patch.
This patch goes towards getting the model boundaries right to transition the architecture towards an event model.
Note: This would theoretically allow us to land auto-rollover, because we don't now pass the correct event-id through. However, we decided that it's better to pull the architecture refactoring through before landing the rollover feature 😬
Co-authored-by: Daniel Karzel <daniel@comit.network>
3 years ago
563a4f3c74
Merge #663
663: Add connected takers feed r=klochowicz a=klochowicz
- add a new watch channel with information about connected takers from the maker's actor system
- test the watch channel behaviour with a unit test
- add a list of connected takers to maker's UI (for testing/monitoring)
This PR will enable us to write more elaborate assertions (e.g. reconnection of the same taker & resuming its operation) and also enables us to gauge the length & amount of inteactions with the maker
Co-authored-by: Mariusz Klochowicz <mariusz@klochowicz.com>
3 years ago
622a8ea696
Refactor multiple uses of `useAsync` into a single `usePostRequest` hook
3 years ago
1e1add8afd
Use projection actor to publish information about connected takers
Note: Projection actor is used across both taker and maker, so it really should
publish information about connected counterparty.
Taker should use that channel for deriving maker online status.
3 years ago
ba9aa11fc1
Display connected takers in the maker's UI
Simple list of connected takers (as the UI is mostly used for testing, it
doesn't need to look pretty).
3 years ago
e27fcce8c5
Provide information about connected takers in a dedicated feed
Publish Vec<TakerId> on a dedicated watch channel and emit it on the event
stream.
Add a test that verifies that the maker updates the feed correctly.
3 years ago
5ee64dc50e
Remove oracle announcement monitoring from before complete rollover
We do this once rollover is complete, not upon accept.
3 years ago
b4b5cd7f57
Move settlement oracle event id into the Dlc
Oracle event-id and Dlc have to exist together, one without the other is incomplete, hence we move the event id inside.
Note that the ID is actually already know to the Dlc as part of the `cet` hashmap, but this was not cleaned up in this patch.
This patch goes towards getting the model boundaries right to transition the architecture towards an event model.
3 years ago
cdf5610ccd
Remove unnecessary `useAsync` layer
We can make further modifications of the body in the event handler,
no need for doing another `useAsync`.
3 years ago
52f5106053
Rename `Hooks.tsx` file to what it contains
1. This does not contain JSX syntax so no need for ts_x_.
2. It default-exports a function, implying that we are not going
to export anything else, thus we can rename it to the hook that we
are exporting.
3 years ago
b8839648d5
Merge #681
681: Allow running taker without maker online r=klochowicz a=klochowicz
See commit messages for details.
Co-authored-by: Mariusz Klochowicz <mariusz@klochowicz.com>
3 years ago
cd3335e20f
Display maker status in the NavBar and an alert if maker is offline
Warn the users that some functionality might be unavailable if there is no maker
online.
Co-authored-by: rishflab <rishflab@hotmail.com>
3 years ago
f503d19824
Allow running taker without connected maker
Instead of blocking and awaiting for connection to maker before the start, spawn
the connection attempt in a separate task that will also attempt to reconnect if
the maker goes offline.
Extend the unit test to cover the new behaviour.
Do not shutdown the taker upon connection loss anymore.
Co-authored-by: rishflab <rishflab@hotmail.com>
3 years ago
ad2ceda3da
Merge #687
687: Implement sqlx::Type for more datatypes r=da-kami a=thomaseizinger
This maker our DB code slightly simpler because we can directly use
our newtypes in the query and they are also constructed right away
by sqlx.
`@da-kami` `@klochowicz` Could one of you run `./prepare-db.sh` locally? It doesn't work on my machine for some reason ...
Co-authored-by: Thomas Eizinger <thomas@eizinger.io>
3 years ago
fe064ba6fe
Merge #685
685: Bump futures from 0.3.17 to 0.3.18 r=thomaseizinger a=dependabot[bot]
Bumps [futures](https://github.com/rust-lang/futures-rs ) from 0.3.17 to 0.3.18.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a href="https://github.com/rust-lang/futures-rs/releases ">futures's releases</a>.</em></p>
<blockquote>
<h2>0.3.18</h2>
<ul>
<li>Fix unusable <code>Sink</code> implementation on <code>stream::Scan</code> (<a href="https://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2499 ">#2499</a>)</li>
<li>Make <code>task::noop_waker_ref</code> available without <code>std</code> feature (<a href="https://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2505 ">#2505</a>)</li>
<li>Add async <code>LineWriter</code> (<a href="https://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2477 ">#2477</a>)</li>
<li>Remove dependency on <code>proc-macro-hack</code>. This raises MSRV of utility crates to 1.45. (<a href="https://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2520 ">#2520</a>)</li>
</ul>
</blockquote>
</details>
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a href="https://github.com/rust-lang/futures-rs/blob/master/CHANGELOG.md ">futures's changelog</a>.</em></p>
<blockquote>
<h1>0.3.18 - 2021-11-23</h1>
<ul>
<li>Fix unusable <code>Sink</code> implementation on <code>stream::Scan</code> (<a href="https://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2499 ">#2499</a>)</li>
<li>Make <code>task::noop_waker_ref</code> available without <code>std</code> feature (<a href="https://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2505 ">#2505</a>)</li>
<li>Add async <code>LineWriter</code> (<a href="https://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2477 ">#2477</a>)</li>
<li>Remove dependency on <code>proc-macro-hack</code>. This raises MSRV of utility crates to 1.45. (<a href="https://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2520 ">#2520</a>)</li>
</ul>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a href="97ee8e6d0967b08562ded5c7ba7a17https://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2521 ">#2521</a>)</li>
<li><a href="c2f4aeac23cc19f6c6ec09b018648beeae63413bd849698e40https://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2477 ">#2477</a>)</li>
<li><a href="c4bd5ce67ahttps://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2505 ">#2505</a>)</li>
<li><a href="f562050ea3https://github-redirect.dependabot.com/rust-lang/futures-rs/issues/2499 ">#2499</a>)</li>
<li>Additional commits viewable in <a href="https://github.com/rust-lang/futures-rs/compare/0.3.17...0.3.18 ">compare view</a></li>
</ul>
</details>
<br />
[](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores )
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting ``@dependabot` rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
<details>
<summary>Dependabot commands and options</summary>
<br />
You can trigger Dependabot actions by commenting on this PR:
- ``@dependabot` rebase` will rebase this PR
- ``@dependabot` recreate` will recreate this PR, overwriting any edits that have been made to it
- ``@dependabot` merge` will merge this PR after your CI passes on it
- ``@dependabot` squash and merge` will squash and merge this PR after your CI passes on it
- ``@dependabot` cancel merge` will cancel a previously requested merge and block automerging
- ``@dependabot` reopen` will reopen this PR if it is closed
- ``@dependabot` close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- ``@dependabot` ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- ``@dependabot` ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- ``@dependabot` ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
</details>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
3 years ago
1e0bf3de72
Implement sqlx::Type for more datatypes
This maker our DB code slightly simpler because we can directly use
our newtypes in the query and they are also constructed right away
by sqlx.
3 years ago
1b6f1e7bc6
Merge #686
686: Only pass the necessary parameters to setup/rollover r=da-kami a=da-kami
Defines the boundaries more clearly which makes it easier when transitioning to the new model.
Co-authored-by: Daniel Karzel <daniel@comit.network>
3 years ago
a3e8427615
Merge #664
664: Withdraw through UI r=bonomat a=bonomat
Enable withdraw through UI.
If the withdraw fails the error messages are being swallowed somewhere and I don't know how to get it. Maybe `@da-kami` or `@thomaseizinger` has an idea?
The HTTP post request fails and I get an `Internal Server Error` but the actual error message is not shown.
I solved the _withdraw all_ feature by saying: if input == 0, withdraw all.
resolves #608
<img width="882" alt="image" src="https://user-images.githubusercontent.com/224613/142815791-da5380d9-52aa-4fcd-9372-fcc5ccc9aefa.png ">
URL: https://mempool.space/testnet/tx/c33de791867148c3fa7e67858b48521a72bd83edd4efe21d0666af26e5939d0d
Co-authored-by: bonomat <philipp@hoenisch.at>
3 years ago
cebe3c0336
Use form and form submit for better handling
3 years ago
c951fe6f57
Add new endpoint to allow withdrawing through UI
3 years ago
500c6a5dfe
Only pass the necessary parameters to setup/rollover
Defines the boundaries more clearly which makes it easier when transitioning to the new model.
3 years ago
4999d3bfa8
Bump futures from 0.3.17 to 0.3.18
Bumps [futures](https://github.com/rust-lang/futures-rs ) from 0.3.17 to 0.3.18.
- [Release notes](https://github.com/rust-lang/futures-rs/releases )
- [Changelog](https://github.com/rust-lang/futures-rs/blob/master/CHANGELOG.md )
- [Commits](https://github.com/rust-lang/futures-rs/compare/0.3.17...0.3.18 )
---
updated-dependencies:
- dependency-name: futures
dependency-type: direct:production
update-type: version-update:semver-patch
...
Signed-off-by: dependabot[bot] <support@github.com>
3 years ago
dcac61f71f
Merge #573
573: Should not be able to trigger close in `PendingOpen` r=scratchscratchscratchy a=scratchscratchscratchy
fixes https://github.com/itchysats/itchysats/issues/517
Triggering a settlement proposal prior to being `Open` does not make sense.
Co-authored-by: scratchy <scratchy@itchysats.network>
3 years ago
825b80d6df
Add guard for triggering settlement proposal
It only makes sense to allow proposing a settlement when in `Open` state.
Theoretically we could argue that it should be allowed in `PendingRefund` and `PendingCommit`, but I don't see an advantage in that.
Note: This only guards the taker side, no guards were added to the maker side because it remains unclear if the deamon should contain logic for this, or we always want to handle acceptance criteria for settlement from the outside.
3 years ago
d49e2efa83
Should not be able to trigger close in `PendingOpen`
Triggering a settlement proposal prior to being `Open` does not make sense.
3 years ago
1fc9ab742e
Merge #666
666: Fixed settlement interval r=da-kami a=da-kami
This work was done as pre-work for #627 which is currently blocked by the model refactor.
I think we should still get these changes in!
Co-authored-by: Daniel Karzel <daniel@comit.network>
3 years ago
5850575327
Fixed settlement interval
A configurable settlement interval for the maker causes friction when implementing auto-rollover, because the settlement interval is not known to the taker.
The settlement interval should not be configurable, thus the configurable settlement interval is removed on the maker side.
The cfd actors and oracle actor use a global constant to decide the settlement interval and oracle even lookahead.
Not pulling the constant into the actors is a conscious decision, it allows us to set this to a different value in the actor tests.
The fact that we use the same constant for cfd actor and oracle actor is an implementation detail.
3 years ago
d853b8ec0b
Move tests to bottom of file
3 years ago
6003ad0bc7
Cleanup impl blocks
3 years ago
a9e4f42f6b
Merge #678
678: We can safely assume that creating a timestamp should not fail r=da-kami a=da-kami
Expect instead of returning a `Result`.
Co-authored-by: Daniel Karzel <daniel@comit.network>
3 years ago
566aca374b
We can safely assume that creating a timestamp should not fail
Expect instead of returning a `Result`.
3 years ago
55dd2fa7d9
Throw HTTP Error so that `userErrorToast` works
3 years ago
d48a39a56f
Callee should take care of the logging
3 years ago
7344a4ee89
The wallet should check if the passed-in address has correct network
3 years ago
2de980b12c
Merge #660
660: Move wallet info out of navbar into main screen r=bonomat a=bonomat
resolves #519
resolves #536
Co-authored-by: bonomat <philipp@hoenisch.at>
3 years ago
090214fd49
Move wallet info out of navbar into main screen
resolves #519
resolves #536
3 years ago
4e39ce4199
Merge #676
676: Bump @vitejs/plugin-react from 1.0.9 to 1.1.0 in /taker-frontend r=thomaseizinger a=dependabot[bot]
Bumps [`@vitejs/plugin-react](https://github.com/vitejs/vite/tree/HEAD/packages/plugin-react )` from 1.0.9 to 1.1.0.
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a href="https://github.com/vitejs/vite/blob/main/packages/plugin-react/CHANGELOG.md "><code>`@vitejs/plugin-react</code>'s` changelog</a>.</em></p>
<blockquote>
<h1><a href="https://github.com/vitejs/vite/compare/plugin-react@1.1.0-beta.1...plugin-react@1.1.0 ">1.1.0</a> (2021-11-22)</h1>
<h1><a href="https://github.com/vitejs/vite/compare/plugin-react@1.1.0-beta.0...plugin-react@1.1.0-beta.1 ">1.1.0-beta.1</a> (2021-11-19)</h1>
<h3>Bug Fixes</h3>
<ul>
<li><strong>plugin-react:</strong> apply <code>babel.plugins</code> to project files only (<a href="https://github-redirect.dependabot.com/vitejs/vite/issues/5255 ">#5255</a>) (<a href="377d0be5cfhttps://github-redirect.dependabot.com/vitejs/vite/issues/5760 ">#5760</a>) (<a href="d93a9fab89https://github-redirect.dependabot.com/vitejs/vite/issues/5761 ">#5761</a>) (<a href="59471b1866https://github-redirect.dependabot.com/vitejs/vite/issues/5669 ">#5669</a>) (<a href="41a7c9ccfbhttps://github.com/vitejs/vite/compare/plugin-react@1.0.6...plugin-react@1.1.0-beta.0 ">1.1.0-beta.0</a> (2021-10-28)</h1>
<h3>Bug Fixes</h3>
<ul>
<li><strong>plugin-react:</strong> avoid mangling the sourcemaps of virtual modules (<a href="https://github-redirect.dependabot.com/vitejs/vite/issues/5421 ">#5421</a>) (<a href="8556ffe3c5https://github.com/vitejs/vite/compare/plugin-react@1.0.5...plugin-react@1.0.6 ">1.0.6</a> (2021-10-25)</h2>
<h3>Bug Fixes</h3>
<ul>
<li><strong>plugin-react:</strong> account for querystring in transform hook (<a href="https://github-redirect.dependabot.com/vitejs/vite/issues/5333 ">#5333</a>) (<a href="13c381368chttps://github.com/vitejs/vite/compare/plugin-react@1.0.4...plugin-react@1.0.5 ">1.0.5</a> (2021-10-18)</h2>
<h3>Bug Fixes</h3>
<ul>
<li><strong>plugin-react:</strong> fix regex for react imports (<a href="https://github-redirect.dependabot.com/vitejs/vite/issues/5274 ">#5274</a>) (<a href="00b3e4fe10https://github-redirect.dependabot.com/vitejs/vite/issues/5314 ">#5314</a>) (<a href="8ce2ea17d5https://github.com/vitejs/vite/compare/plugin-react@1.0.3...plugin-react@1.0.4 ">1.0.4</a> (2021-10-11)</h2>
<h2><a href="https://github.com/vitejs/vite/compare/plugin-react@1.0.2...plugin-react@1.0.3 ">1.0.3</a> (2021-10-11)</h2>
<!-- raw HTML omitted -->
</blockquote>
<p>... (truncated)</p>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a href="011552210dcb9f750e88https://github.com/vitejs/vite/commits/create-app@1.1.0/packages/plugin-react ">compare view</a></li>
</ul>
</details>
<br />
[](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores )`
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting ``@dependabot` rebase`.
[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)
---
<details>
<summary>Dependabot commands and options</summary>
<br />
You can trigger Dependabot actions by commenting on this PR:
- ``@dependabot` rebase` will rebase this PR
- ``@dependabot` recreate` will recreate this PR, overwriting any edits that have been made to it
- ``@dependabot` merge` will merge this PR after your CI passes on it
- ``@dependabot` squash and merge` will squash and merge this PR after your CI passes on it
- ``@dependabot` cancel merge` will cancel a previously requested merge and block automerging
- ``@dependabot` reopen` will reopen this PR if it is closed
- ``@dependabot` close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- ``@dependabot` ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- ``@dependabot` ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- ``@dependabot` ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
</details>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
3 years ago
ec23aadd49
Merge #645
645: Introduce projection actor r=bonomat a=bonomat
Resolves #572
Co-authored-by: bonomat <philipp@hoenisch.at>
3 years ago
bf36f290dd
Introduce projection actor
The projection actor is responsible in preparing data for the HTTP API and consequently for the UI.
While this is commit provides only the foundation, in the long run we can:
- Reduce the logic happening in the rocket layer. `ToSseEvent` can likely go away.
- Reduce the complexity for other actors in what needs to be updated when. All they should care about is sending updates to the projection actor. (mostly done)
- Improve test coverage. With a dedicated actor that does the projection, we should be able to write assertions in our integration tests that are closer to the UI.
3 years ago
1d6b1a49e1
This test did not test much
3 years ago
7d98e04686
Move invariant into function
This invariant holds for all CFDs which get inserted into the DB, hence, we can move it inside the insert function.
3 years ago
c2f61ef79b
Bump @vitejs/plugin-react from 1.0.9 to 1.1.0 in /taker-frontend
Bumps [@vitejs/plugin-react ](https://github.com/vitejs/vite/tree/HEAD/packages/plugin-react ) from 1.0.9 to 1.1.0.
- [Release notes](https://github.com/vitejs/vite/releases )
- [Changelog](https://github.com/vitejs/vite/blob/main/packages/plugin-react/CHANGELOG.md )
- [Commits](https://github.com/vitejs/vite/commits/create-app@1.1.0/packages/plugin-react )
---
updated-dependencies:
- dependency-name: "@vitejs/plugin-react"
dependency-type: direct:production
update-type: version-update:semver-minor
...
Signed-off-by: dependabot[bot] <support@github.com>
3 years ago
faad0cc816
Merge #628
628: Misc cleanup r=da-kami a=thomaseizinger
Co-authored-by: Thomas Eizinger <thomas@eizinger.io>
3 years ago
dependabot[bot]
bors[bot]
Thomas Eizinger
Mariusz Klochowicz
Daniel Karzel
bonomat
scratchy
