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

only allow one instance of Frame, focus instance when trying to run a… #927

Merged
merged 7 commits into from
Jul 11, 2022

Conversation

mholtzman
Copy link
Collaborator

…nother instance

  • using Electron single instance lock to avoid errors when a second instance is started
    • quitting the second instance
    • focusing the running instance
  • logging the request for a second instance

@goosewobbler goosewobbler force-pushed the one-instance branch 2 times, most recently from 911b2b0 to b0981e0 Compare July 7, 2022 01:07
main/index.js Outdated Show resolved Hide resolved
main/index.js Show resolved Hide resolved
main/index.js Outdated Show resolved Hide resolved
main/index.js Outdated Show resolved Hide resolved
@mholtzman mholtzman merged commit a70a1dd into beta.21 Jul 11, 2022
@mholtzman mholtzman deleted the one-instance branch July 11, 2022 16:52
mholtzman added a commit that referenced this pull request Jul 12, 2022
* beta.21

* dont allow undefined 'to' field in transactions (#921)

* get rid of inputref focus error

* update package lock version

* upgrade GridPlus SDK to v2.1.0 (#925)

* return not connected if no connection is found for requested chain (#924)

* sentry asar path stripping (#922)

Co-authored-by: goosewobbler <goosewobbler@pm.me>

* Worker process (#934)

* add infrastructure for worker processes

* add name

* test

* finish tests

* IPFS getFile connection error fix (#932)

Co-authored-by: goosewobbler <goosewobbler@pm.me>

* limit sentry events (#930)

* only allow one instance of Frame, focus instance when trying to run another instance

* improve readability

* add log for requested second instance

* remove capital

* sentry => errors, move exception handling

* fix tests

* limit sentry events

* update for event rate limiting

* tweak names

* rename sentry to errors, move uncaughtExceptionHandler

* Update test/main/sentry/index.test.js

Co-authored-by: Matt Holtzman <matt.holtzman@gmail.com>

* address review comments

* add dialog import back

* move captureException to correct place

* Update index.js

* Update index.js

* Update index.js

* Update index.test.js

Co-authored-by: Matt Holtzman <matt.holtzman@gmail.com>
Co-authored-by: goosewobbler <goosewobbler@pm.me>

* only allow one instance of Frame, focus instance when trying to run a… (#927)

* only allow one instance of Frame, focus instance when trying to run another instance

* improve readability

* add log for requested second instance

* remove capital

* app.exit

* remove return

Co-authored-by: goosewobbler <goosewobbler@pm.me>
Co-authored-by: goosewobbler <432005+goosewobbler@users.noreply.github.com>

* Update index.test.js

* Update index.js

* remove captureException

* Use domains to handle worker updates (#929)

* create worker process abstraction, use it for updates

* update timeouts

* finish worker abstractions

* cleanup

* revert timeout delay

* use spawn instead of fork

* dismiss window properly

* address comments, make some methods private

* add version display, move store access out of updater, add logging

* update to not use electron in child process

* add platform specific constructors

* clean up whitespace, finish rebase

* finish implementing electron app in auto updater

* use domains

* temp commit

* get rid of dev stuff

* cleanup

* add error type

* reset interval

* PR fixes

* Gridplus sdk 2.2.0 (#936)

* Bumps gridplus-sdk to v2.2.0

* update package lock

Co-authored-by: Doug Lance <doug.lance@gmail.com>

* add path test, minor cleanup

* added missing latestRelease

* removed stray comma

Co-authored-by: goosewobbler <goosewobbler@pm.me>
Co-authored-by: goosewobbler <432005+goosewobbler@users.noreply.github.com>
Co-authored-by: Doug Lance <doug.lance@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants