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

fix(datasets): dataset_name changed to short_name #913

Merged
merged 1 commit into from Apr 24, 2020

Conversation

vfried
Copy link
Contributor

@vfried vfried commented Apr 22, 2020

"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)

@vfried vfried requested a review from a team as a code owner April 22, 2020 07:50
@vfried vfried force-pushed the 907-dataset-short-name branch 2 times, most recently from f5e7351 to bed6afd Compare April 22, 2020 08:34
Copy link
Contributor

@ciyer ciyer left a 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

vfried added a commit that referenced this pull request Apr 22, 2020
BREAKING_CHANGE: dataset_name changed to short_name in the backend (renku-core)
@vfried
Copy link
Contributor Author

vfried commented Apr 22, 2020

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

Done!

vfried added a commit that referenced this pull request Apr 23, 2020
BREAKING_CHANGE: dataset_name changed to short_name in the backend (renku-core)
BREAKING_CHANGE: dataset_name changed to short_name in the backend (renku-core)
Copy link
Contributor

@ciyer ciyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@vfried vfried merged commit dce74cd into master Apr 24, 2020
@ciyer ciyer added this to the sprint-2020-03-27 milestone May 7, 2020
@ciyer ciyer deleted the 907-dataset-short-name branch June 29, 2020 21:31
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