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

Remove cloudshell #330

Merged
merged 1 commit into from
Dec 19, 2023
Merged

Conversation

johnksv
Copy link
Contributor

@johnksv johnksv commented Dec 18, 2023

After discussion on Slack ( https://3innovation.slack.com/archives/C04139Q26TF/p1702736322262879 ). Cloudshell was an old feature present in the early versions of Onyxia that was similar to the cloudshell feature you have on GCP or other cloud providers. It is not on the roadmap to reintroduce it.

Also update swagger description on types to be empty (as most other types). I found it kind of confusing that the description was Cloudshell data and health

After discussion on Slack. Cloudshell was an old feature present in the early versions of Onyxia that was similar to the cloudshell feature you have on GCP or other cloud providers.
It is not on the roadmap to reintroduce it.
@olevitt
Copy link
Contributor

olevitt commented Dec 19, 2023

Great, thanks 👍

@olevitt olevitt merged commit 06ad2ad into InseeFrLab:main Dec 19, 2023
3 checks passed
@johnksv johnksv deleted the remove-cloudshell branch December 20, 2023 10:12
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