-
Notifications
You must be signed in to change notification settings - Fork 6
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
fix(datasets): dataset_name changed to short_name #913
Conversation
f5e7351
to
bed6afd
Compare
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.
Code makes sense. Could you just add a reference to the PR and issue in the commit message? And could you add that there is a breaking change here, since this needs a new version of a backend component?
https://www.conventionalcommits.org/en/v1.0.0/#summary
bed6afd
to
4301dce
Compare
BREAKING_CHANGE: dataset_name changed to short_name in the backend (renku-core)
4301dce
to
e205edd
Compare
Done! |
BREAKING_CHANGE: dataset_name changed to short_name in the backend (renku-core)
e205edd
to
8ee6d83
Compare
BREAKING_CHANGE: dataset_name changed to short_name in the backend (renku-core)
8ee6d83
to
4ddfa12
Compare
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.
👍
"dataset_name" inside the core service changed to "short_name" this will allow us to use in the future for "dataset_name" a proper title with spaces and short name would be the title that we have now.
For testing this in your own env you need to have the latest version of renku-core(0.10.3-cf08d03)