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: chrome tab open failed due to chrome process not running #1772

Merged
merged 1 commit into from Nov 11, 2023

Conversation

YumengNevix
Copy link
Contributor

Connection with issue(s)

Resolve issue #1771 #1771

Connected to #???

Testing and Review Notes

Screenshots or Videos

To Do

  • double check the original issue to confirm it is fully satisfied
  • add testing notes and screenshots in PR description to help guide reviewers
  • request the "UX" team perform a design review (if/when applicable)

@pichillilorenzo
Copy link
Owner

Thanks! By the way, I think it could be better to use another boolean value instead of setting intialUrl to null to avoid webpage reopen.

@pichillilorenzo pichillilorenzo merged commit 828cc7d into pichillilorenzo:master Nov 11, 2023
@pichillilorenzo
Copy link
Owner

@all-contributors please add @YumengNevix for code

Copy link
Contributor

@pichillilorenzo

I've put up a pull request to add @YumengNevix! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants