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

Renaming base fields #1466

Merged
merged 1 commit into from Oct 9, 2019
Merged

Renaming base fields #1466

merged 1 commit into from Oct 9, 2019

Conversation

fao89
Copy link
Member

@fao89 fao89 commented Oct 2, 2019

_id to pulp_id
_created to pulp_created
_last_updated to pulp_last_updated
_href to pulp_href

ref #5457
https://pulp.plan.io/issues/5457

Required PR: pulp/pulpcore#317
Required PR: pulp/pulpcore-plugin#137
Required PR: pulp/pulp-smash#1220

@fao89 fao89 requested a review from a team October 2, 2019 20:27
@fao89 fao89 changed the title Renaming base fields WIP - Renaming base fields Oct 3, 2019
@fao89 fao89 changed the title WIP - Renaming base fields Renaming base fields Oct 3, 2019
@@ -0,0 +1 @@
Change `_id`, `_created`, `_last_updated`, `_href` to `pulp_id`, `pulp_created`, `pulp_last_updated`, `pulp_href`
Copy link
Member

Choose a reason for hiding this comment

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

I think it's probably .removal and not .feature because we are changing api and it's a backward incompatible change

@fao89 fao89 force-pushed the 5457 branch 4 times, most recently from d0b69a9 to 4ac5928 Compare October 7, 2019 14:33
_id to pulp_id
_created to pulp_created
_last_updated to pulp_last_updated
_href to pulp_href

ref #5457
https://pulp.plan.io/issues/5457

Required PR: pulp/pulpcore#317
Required PR: pulp/pulpcore-plugin#137
Required PR: pulp/pulp-smash#1220
@daviddavis daviddavis merged commit f860494 into pulp:master Oct 9, 2019
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

3 participants