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

Poll wk version to notify during upgrade #6451

Merged
merged 4 commits into from
Sep 29, 2022
Merged

Conversation

fm3
Copy link
Member

@fm3 fm3 commented Sep 8, 2022

URL of deployed dev instance (used for testing):

  • https://___.webknossos.xyz

Steps to test:

  • open help menu in navbar
  • check in dev tools that every 2s a request is made to build info
  • close the menu
  • --> requests should stop
  • update the server (or monkeypatch it somehow) --> a message should appear in the help menu (I tested this and it worked)

  • Ready for review

@philippotto philippotto marked this pull request as ready for review September 28, 2022 12:02
@philippotto
Copy link
Member

@fm3 I added some small changes and think the code is fine now :) do you want to test this, too?

@philippotto philippotto changed the title [WIP] Poll wk version to notify during upgrade Poll wk version to notify during upgrade Sep 28, 2022
@fm3
Copy link
Member Author

fm3 commented Sep 28, 2022

If it works for you, I think this is good to go. This can’t really break any existing functionality and I assume you tested it

@philippotto philippotto enabled auto-merge (squash) September 29, 2022 08:17
@philippotto philippotto merged commit 68f68ad into master Sep 29, 2022
@philippotto philippotto deleted the version-polling branch September 29, 2022 08:38
hotzenklotz added a commit that referenced this pull request Oct 13, 2022
…jects-created

* 'master' of github.com:scalableminds/webknossos: (337 commits)
  Fix docs for the annotation download file format (#6546)
  Added total runtime information to VX reports (#6543)
  fix VX report for completed + skipped tasks (#6540)
  Avoid allocating spire uint objects during apply agglomerate (#6532)
  Explore remote N5 datasets (#6520)
  Fix MeshChunk byteOffset (Long, not Int) (#6536)
  update browserslist (#6505)
  Support new Mesh File (v3) (#6491)
  makes workflow_yamlContent optional (#6518)
  Always return 404 for Failures in Zarr Streaming (#6515)
  Poll wk version to notify during upgrade (#6451)
  add script which extracts newest changelog and creates GH release for it (#6504)
  release 22.10.0 (#6500)
  voxel³ -> voxel (#6501)
  Allow task type summary to identify task type when creating tasks in bulk (#6486)
  Fix sql evolution 090 (defer not null constraint) (#6498)
  SQL schema cleanup (#6492)
  Fix validation of layer selection when trying to start globalization of floodfills (#6497)
  Add "shift + w" shortcut to cycle backwards through tools (#6493)
  Fix filtering for public datasets in dataset table (#6496)
  ...
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.

2 participants