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.

Sign up for GitHub

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

Jump to bottom

Settings should throw an error when attempting to store non-serializable data #946

Open
jsalankey opened this issue Apr 22, 2020 · 0 comments
Open

Settings should throw an error when attempting to store non-serializable data #946

jsalankey opened this issue Apr 22, 2020 · 0 comments

Comments

@jsalankey
Copy link
Contributor

Settings currently fails in a mostly invisible way if you attempt to save a function or other non-serializable thing to it. It should either log or throw an error when this happens as it's very easy to miss.
smithtb pushed a commit to smithtb/opensphere that referenced this issue Oct 7, 2020
@justin-bits
Merge pull request ngageoint#946 in WV/opensphere from ~DENTONJ/opens…
6943b2c 
…phere:CATS-91 to master

* commit '91aa351f3b36c1659cec7866c9239def481dfcc6':
  test(cypress): fixes automatic termination of webserver under Windows
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jsalankey