feat: add notebook to upgrade shared tier #136
Conversation
j-fitas
force-pushed
the
MCDB-75127/add-upgrade-notebook
branch
from
c75dcc2 to
d81043b
Compare
| @@ -17,7 +17,7 @@ | |||
| ] | |||
| }, | |||
| { | |||
| "id": "bd0ae268", | |||
There was a problem hiding this comment.
I want to run this by @kesmit13, because its one of the pre-commit hooks that changes that. I might be using it wrong.
| tags=["starter", "notebooks", "fusion"] | ||
| lesson_areas=[] | ||
| destinations=["spaces"] | ||
| minimum_tier="free-shared" |
There was a problem hiding this comment.
Shouldn't the minimum tier be trial or standard, since this operation cannot be performed on the free tier?
There was a problem hiding this comment.
Trial is not an option for this field, we use free-shared when we want something available on trial. Or can bump it to Standard and not make it available for Trial.
| "cell_type": "markdown", | ||
| "metadata": {}, | ||
| "source": [ | ||
| "<div class=\"alert alert-block alert-warning\">\n", |
There was a problem hiding this comment.
There's something weird about having two consecutive "Alert" blocks in the notebook. Is it possible to merge them?
Also, I was confused about the difference between the two alerts. Aren't they saying the same?
There was a problem hiding this comment.
We can combine them, yes.
The first one is the default indication on what this notebook can be ran. (Also, just realized that the second sentence is wrong, ill have to remove that)
The second one's purpose is to state that while you can run this on a shared tier you need to be able of creating a workspace group.
There was a problem hiding this comment.
@Rodriguespn, pls check the lastest commit.
Yes, I'll have to get either the UX or docs team involve to polish this experience. |
Add notebook to upgrade shared tier vws to dedicated workspace groups
https://memsql.atlassian.net/browse/MCDB-75127