animo / siera-desktop

https://siera.animo.id/desktop
8 stars 3 forks source link

refactor: auto accept created invitation #98

Closed TimoGlastra closed 1 year ago

TimoGlastra commented 1 year ago

This PR removes the accept button from the connection list when you receive a request for an invitation you created. I think most wallets expect a connection to be completed immediately (it's just a cryptographic relation).

If not desired, feel free to close this PR, but wanted to tinker a bit with Siera.

We may want to look at the flickering of the states as that now happens almost immediately (see video)

https://user-images.githubusercontent.com/23165168/224819530-1e89c9dd-0b59-4f74-8dce-709393f45cb8.mov

codecov-commenter commented 1 year ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@d318ef8). Click here to learn what that means. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #98   +/-   ##
=======================================
  Coverage        ?   76.74%           
=======================================
  Files           ?       11           
  Lines           ?       43           
  Branches        ?        3           
=======================================
  Hits            ?       33           
  Misses          ?       10           
  Partials        ?        0           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more