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

Rename resource names #189

Closed
wants to merge 2 commits into from
Closed

Conversation

erichuanggit
Copy link
Collaborator

To be consistent with other end-points, rename below three ones:

  • content-delivery-content-category -> content-delivery-content-categories
  • content-delivery-content-format -> content-delivery-content-formats
  • content-delivery-service -> content-delivery-services
    The old names remain available for backwards compatibility, but has been
    marked deprecated.

JIRA: PDC-1027

@simozhan
Copy link
Contributor

looks good to me

@lubomir
Copy link
Member

lubomir commented Oct 29, 2015

I think it would be nice to change the router base_names for the deprecated end-points. With the patch there are duplicate base_names in there, and it may not cause problems, but it is confusing for readers of the code.

@erichuanggit
Copy link
Collaborator Author

@lubomir Very nice proposal. Since the deprecated end-point will be remove in future. I did not think about more.

To be consistent with other end-points, rename below three ones:
* content-delivery-content-category -> content-delivery-content-categories
* content-delivery-content-format -> content-delivery-content-formats
* content-delivery-service -> content-delivery-services
* persons -> contacts/people
* maillists -> contacts/mailing-lists
The old names remain available for backwards compatibility, but has been
marked deprecated.

JIRA: PDC-1027
@lubomir
Copy link
Member

lubomir commented Nov 2, 2015

Looks fine.

@erichuanggit
Copy link
Collaborator Author

@lubomir Thanks. deprecated should be the right one, will fix.

@ycheng-aa
Copy link
Contributor

Looks good.

@erichuanggit erichuanggit deleted the PDC-1027_Rename-resource-names branch November 4, 2015 04:14
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

4 participants