-
Notifications
You must be signed in to change notification settings - Fork 511
Rename workspace to project #3407
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
Conversation
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
327b723
to
e15d612
Compare
workspace_table.c.id, | ||
workspace_table.c.name, | ||
) | ||
): |
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.
Should we also ensure that existing workspace names are lowercase and have no special characters?
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 has a near-zero chance of happening. I already blocked the DB upgrade in the previous PR if there is more than one workspace present. And it's very improbable that someone would actually use a weird name for their default workspace, given that it wasn't possible to manage them with the CLI and dashboard before.
) | ||
batch_op.drop_column("workspace_id") | ||
|
||
with op.batch_alter_table("user", schema=None) as batch_op: |
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.
We drop this but don't seem to add a default_project_id
as replacement?
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.
You're right. I don't really need this anymore. It was meant as a temporary measure for a quicker PoC, but now we decided to go with a fully updated dashboard which supports creating/switching between workspaces, so I dropped it.
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.
I thought we maybe wanted to even have that as an option for the user to configure in the dashboard, to essentially select which project will be active by default each time they access a workspace. But I guess when we get to that point we can always implement that
display_name = data.get("display_name") | ||
|
||
if not name and not display_name: | ||
return data |
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 will trigger a validation error which i guess is intended?
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.
Yes 😄 , I just pass it to Pydantic here.
fb226ec
to
e052bd9
Compare
Describe changes
This PR is the final phase in implementing ZenML projects: the big rename from workspace to project and introducing "cute names for projects".
Renaming changes:
workspace
DB table toproject
workspace_id
foreign key relationships toproject_id
in the DBworkspace
toproject
everywhere else: the domain models, the Zen stores, the Client, the CLI and the REST API endpointsworkspace
endpoints to preserve backwards-compatibility for the dashboard"Cute names" for projects:
Pre-requisites
Please ensure you have done the following:
develop
and the open PR is targetingdevelop
. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.Types of changes