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

fix: connect status is always connected #157

Merged
merged 8 commits into from
Mar 28, 2023

Conversation

IronLu233
Copy link
Contributor

What Changed

closes #149

Motivation

In previous version, the connect status indicator can not react when tab changes.
This PR fix it.

image
image

Change Type

Indicate the type of change your pull request is:

  • documentation
  • patch
  • minor
  • major

@IronLu233 IronLu233 marked this pull request as ready for review March 27, 2023 05:43
@codecov
Copy link

codecov bot commented Mar 27, 2023

Codecov Report

Merging #157 (b530c4c) into main (5543361) will not change coverage.
The diff coverage is 0.00%.

❗ Current head b530c4c differs from pull request most recent head 5e2d396. Consider uploading reports for the commit 5e2d396 to get more accurate results

@@           Coverage Diff           @@
##             main     #157   +/-   ##
=======================================
  Coverage   87.25%   87.25%           
=======================================
  Files          48       48           
  Lines         816      816           
  Branches       97       97           
=======================================
  Hits          712      712           
  Misses         21       21           
  Partials       83       83           
Impacted Files Coverage Δ
...on-chrome/__tests__/helpers/mockPlatformService.ts 88.88% <0.00%> (-11.12%) ⬇️

... and 2 files with indirect coverage changes

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

@ckb-js ckb-js deleted a comment from homura Mar 28, 2023
@homura homura changed the title fix: connect status is always Connected fix: connect status is always connected Mar 28, 2023
@homura homura added the patch Increment the patch version when merged label Mar 28, 2023
@homura homura merged commit d72817b into ckb-js:main Mar 28, 2023
@github-actions
Copy link

🚀 PR was released in v0.0.5 🚀

@github-actions github-actions bot added the released This issue/pull request has been released. label Mar 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

connect status is always Connected
2 participants