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

Code upto v0.2.x #17

Open
wants to merge 63 commits into
base: chemotion-cli
Choose a base branch
from
Open

Conversation

harivyasi
Copy link
Contributor

See CHANGELOG.md

Mehmood Ghaffar and others added 30 commits July 26, 2022 11:42
Merge pull request #3 from mehmood86/command_ping
-this commit temporarily breaks `upgrade` till next release
now include compose file in release
- need for update is done by doing version check
- version check is done by resolving online URL
- version check is done only if 24 hours have elapsed since last check
Merge pull request #4 from mehmood86/console_cmd
- remove support for .env file -- needs to reimplemented as an entry in `chemotion-cli.yml` file
- install and new download the latest compose file
- @ptrxyz 's compose file is left as it is -- only the `ports` entry is removed
- additional field accessAddress stores the URL to access ELN as it was given
- status now shows container ids
- instance upgrade now only replaces existing compose file with new one
- bugfix in ping: now also displays the error message received
- cli related settings are now in an additional cli.docker-compose.yml file
- updated to go1.19
rational: it depends on the `selected` instance
- we no longer delete the ports from docker-compose.yml
- only modify it to be `port`:4000 or
- whatever was the previous value when upgrading
- therefore docker-compose.yml is largely as it is downloaded
- we check if docker-compose.cli.yml exists
- if not it will be created on the fly
move from 0.1.x to 0.2.0 happens automagically now
bugfix: instance name should not have period
reimplent mkdir shared in instance if not there
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.

1 participant