-
Notifications
You must be signed in to change notification settings - Fork 1
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
Dev #56
base: main
Are you sure you want to change the base?
Dev #56
Conversation
for more information, see https://pre-commit.ci
…-register Review and add to toc
Aiidalab app registration
* [pre-commit.ci] pre-commit autoupdate updates: - [github.com/psf/black: 22.10.0 → 22.12.0](psf/black@22.10.0...22.12.0) * [pre-commit.ci] pre-commit autoupdate updates: - [github.com/pre-commit/mirrors-prettier: v3.0.0-alpha.4 → v3.0.0-alpha.6](pre-commit/mirrors-prettier@v3.0.0-alpha.4...v3.0.0-alpha.6) - [github.com/psf/black: 22.12.0 → 23.3.0](psf/black@22.12.0...23.3.0) --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Pablo de Andres <9430234+pablo-de-andres@users.noreply.github.com>
* [pre-commit.ci] pre-commit autoupdate updates: - [github.com/psf/black: 22.10.0 → 22.12.0](psf/black@22.10.0...22.12.0) * Update images and remove app linking from process --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@@ -3,7 +3,7 @@ | |||
|
|||
from urllib.request import urlopen | |||
|
|||
api_version = "0.4.0" | |||
api_version = "0.6.0" | |||
version = "1.8.4-beta" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be updated to reflect the latest version of the platform. Potentially even the next one, if you have included information that still have to be deployed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
By latest version of the platform, you mean the latest version of standard-app? If yes, then the ltest one to be deployed is 0.6.0 where I have commited all the changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, I mean latest version of the platform (currently 1.8.6). The docs need to specify with which version they are compatible.
Notice I commented on line 7, not 6
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I will do that. I will update with the next platform version.
No description provided.